- Open file
- On the second line of text in the red box, select "RIA · MEL".
- Change the font size
The black is then stripped from all of the black text on that line (even the stuff not selected).
Well, you've certainly opened up a can of worms here. Attached are three independent patches:
bug_402274_v2.patch is a repost of the same file I attached to my earlier e-mail, which fixes the crash
bug-402274-part-2.patch fixes the problem you spotted here where altering the style of one bit of text would cause something else to change as well
bug-402274-part-3.patch is another problem I spotted when testing part 2 where some simplifications of the XML tree were missed
The first patch definitely needs to go in to this release. The second part is certainly desirable, however I wonder whether it crosses the threshold of seriousness for this late stage of the release, given that the bug must have existed for ages and alters some code which is used in just about every part of text styling (it did seem obviously wrong to me, however). The third part is actually more important than it sounds, because without it repeatedly re-styling overlapping bits of text will create an ever-deepening XML tree until the limit of each character being in its own tspan is reached.
My guess is that bits of code in the second and third parts used to work because they mirrored an ordering bug either in the style parsing or XML attribute parsing code. Does anybody remember fixing such a bug in either of those areas, and if so, when?
Looks like it's time for the release wardens to make some hard decisions. If you need any more information on the possible impact of each patch, please ask.
Richard.