.className = 'subject'
3) At this point I'm satisfied we're looking at the subject , so just grab every bit of text contained within it.
Before it was a good deal more complicated, and, as we have discovered, prone to easy breakage when changes happen to the interface for whatever reason:
code.nkinka.de Git - roundcube.git/commitdiff (http://code.nkinka.de/gitweb?p=roundcube.git;a=commitdiff;h=89320766d84786de4815fb5174dc8473dfa6a919)
Nathan
many thanks nkinkade
i have fixed in my version the thread view for viewing on all browsers the original code works only on opera correctly i have added the patches for /js/app.js /js/list.js and /steps/mail/func.inc to fix displaying errors for all browsers my roundcube is little specialy roundcube svn + roundcube delvel api and many self made patches and features i hope the patch works for you
when user in thread viewing and mark many messages with collapsed messages the collapsed messages are displaying in the moving box here is a litte modification to fix this and not show hidden messages in moving box.
//search to subject = node.nodeType==3 ? node.data : node.innerHTML; names += (subject.length > 50 ? subject.substring(0, 50)+'...' : subject) + '<br />';
//and replace with
if((pnode = obj.childNodes[i].parentNode) && (pnode.nodeName=='TR' && pnode.style.display!='none')) { subject = node.nodeType==3 ? node.data : node.innerHTML; names += (subject.length > 50 ? subject.substring(0, 50)+'...' : subject) + '<br />'; }
sorry for this bad english i'm german ;)
here is little screenshot to see what i mean is a firefox screen
|