Email Threads: unable to style threaded email rows.

[youds]

Hi

Please see: ​http://paste2.org/p/871014

The problem is purely from a CSS/HTML perspective when creating a skin. It is impossible to style emails which appear in a threaded conversation as the only difference between an email in a threaded conversation and a normal email is an additional <span /> in td.subject.
The tr.message element needs to have a class set upon it indicating the message is part of a thread and the level that the email should be displayed at. That way all the td.from, td.threads, td.date and td.attachment elements can be style'ed appropriately.

Thanks

[youds]

Will it be possible to find out what changes will be made so I can prepare my skin for the 0.4-stable release?

Kind regards

[alec]

"child" class for non-root rows (TR) will be enough or we should also add "levelX" class to each row? What do you expect?

[youds]

"levelX" will be necessary because the HTML remains linear. Ideally you would expect the HTML of level1 to contain the html for level2, thus then being able to only include a "child" class for the parent row. However here you have to express it in CSS.
There is another problem to consider, RoundCube includes "even" and "odd" child classes on the messagelist. This means when a root row is closed and it contains an odd number of child messages the styling breaks. When in thread mode, the child messages should be excluded from the root rows "even" / "odd" sequence and begin their own sequence with the root row as the starting point for the child rows.

[alec]

Zebra class has been already removed. I also think that styling threads isn't so important. Let's keep it simple. You can write a plugin.