0 Members and 8 Guests are viewing this topic.
Does it mean we can now use OmnomIRC on a calculator??
function drawChannels(){ "use strict"; var table, td, span, span2, chanName, docfrag = document.createDocumentFragment(), chanList = document.getElementById("ChanList"); for (var i in channels){ // build elements/variables table = document.createElement('table'); td = document.createElement('td'); span = document.createElement('span'); span2 = document.createElement('span'); table.className = 'chanList'; chanName = base64.decode(channels[i]); // Set properties/events td.id = chanName; td.className = (getChannelIndex()==i)?"curchan" :"chan" if (chanName.substr(0,1) != "#"){ span.onclick = (function(name){ return function(){ partChannel(name); } })(chanName); span.onmouseover = function(){ this.style.color = '#C73232'; this.style.fontWeight = 'bolder' }; span.onmouseout = (function(color){ return function(){ this.style.color = color; this.style.fontWeight = 'normal'; } })((getChannelIndex()==i)?'#FFF':'#22C'); span.innerHTML = 'x'; td.appendChild(span); } span2.onclick = (function(i){ return function(){ selectChannel(i); }; })(i); span2.innerHTML = chanName; // Append to DOM td.appendChild(span2); table.appendChild(td); docfrag.appendChild(table); } chanList.innerHTML = ''; chanList.appendChild(docfrag);}
Does Eeems editing it means Opera support is removed?
Aaaand forked to re-write all the JavaScript to follow better coding standards.https://github.com/Eeems/OmnomIRC
Quote from: Eeems on March 23, 2013, 02:00:45 amAaaand forked to re-write all the JavaScript to follow better coding standards.https://github.com/Eeems/OmnomIRCEeems, in your re-written js code you removed all the parts with omnimagaUserId which are important by now for checking if the user is admin for ops.So I can't merge the project at this point, sorry
Quote from: Sorunome on March 23, 2013, 12:41:56 pmQuote from: Eeems on March 23, 2013, 02:00:45 amAaaand forked to re-write all the JavaScript to follow better coding standards.https://github.com/Eeems/OmnomIRCEeems, in your re-written js code you removed all the parts with omnimagaUserId which are important by now for checking if the user is admin for ops.So I can't merge the project at this point, sorrylook at when I forked it. that feature wasn't in it yet. plus, its not ready to be merged. its not done yet. there is a ton of work still to be done.
Quote from: Eeems on March 23, 2013, 01:10:03 pmQuote from: Sorunome on March 23, 2013, 12:41:56 pmQuote from: Eeems on March 23, 2013, 02:00:45 amAaaand forked to re-write all the JavaScript to follow better coding standards.https://github.com/Eeems/OmnomIRCEeems, in your re-written js code you removed all the parts with omnimagaUserId which are important by now for checking if the user is admin for ops.So I can't merge the project at this point, sorrylook at when I forked it. that feature wasn't in it yet. plus, its not ready to be merged. its not done yet. there is a ton of work still to be done. Oh, ok, i have to get more used to github. So can i change other files and merge and those other files stay changed?