Created attachment 194186 [details] Text with overline and underline (odt) Description: If you enable overline for a line of text, overline is also shown (incorrectly) for end of line character. This only happens when you enable showing the formatting marks. Steps to Reproduce: 2. Open attachment, which is an ODT document 3. Enable unprintable characters with: "Toggle Formatting Marks (Ctrl+F10)" button. Actual Results: End of line character, which is a formatting mark is incorrectly overlined in blue. If you disable formatting marks with "Toggle Formatting Marks (Ctrl+F10)", it goes away. This is also true for line break, which is displayed in incorrectly small size. Expected Results: End of line character should not be overlined. User Profile Reset: No Additional Info: Reproducible with LO 24.2: Version: 24.2.2.2 (X86_64) / LibreOffice Community Build ID: d56cc158d8a96260b836f100ef4b4ef25d6f1a01 CPU threads: 12; OS: Linux 6.2; UI render: default; VCL: gtk3 Locale: en-US (en_US.UTF-8); UI: en-US Calc: threaded Also reproducible with the latest LO 24.8 dev master: Version: 24.8.0.0.alpha1+ (X86_64) / LibreOffice Community Build ID: 551c8230b495e8c0e0502b73445f6edd1510633e CPU threads: 12; OS: Linux 6.2; UI render: default; VCL: gtk3 Locale: en-US (en_US.UTF-8); UI: en-US Calc: CL threaded
Confirmed. Weird in that while the Para mark has always followed the font of the paragraph, the other 'Font Effects' overline, strikethrough, and underline have changed. From LO 3.3.0 all font effects were applied to paragraph end mark. Through at least the 6.4.7 release. But by the 7.3.6 release the strikethrough and underline were removed and the overline remained. Don't have 7.0, 7.1 or 7.2 build handy to test. But a change in there somewhere that didn't get applied to the overline. Needs a reverse bibisect to find that change, then see if the same handling can be applied to the overline effect.
Also, why does the New Line (<Shift>+<Enter> line break with downwards arrow leftwards, looks like U+21B5) *not* get the font size at its position in the paragraph text run? Nor does it get any of the character font effects. What's different for that NPC?
(In reply to V Stuart Foote from comment #2) > Also, why does the New Line (<Shift>+<Enter> line break with downwards arrow > leftwards, looks like U+21B5) *not* get the font size at its position in the > paragraph text run? Nor does it get any of the character font effects. > What's different for that NPC? This is another bug, that I will file. And thanks for confirming. Also, one thing to mention is that any solution for this bug should not just copy the way underline is rendered, because in most cases, overline is better (and correctly) rendered, and underline is not. See tdf#155557 comment 10. Paragraph mark is an exception.
Created attachment 194191 [details] zip Test file with screen clips 7.0.6 - 24.8.0 Handling changed in the 7.1 release, between 7.0.6.2 and 7.1.8.1 Also, look at the final para marker before end of page. It picks up the strikethrough and the underline, just like the diminutive new line (<Shift>+<Enter>) marker.
(In reply to V Stuart Foote from comment #1) > From LO 3.3.0 all font effects were applied to paragraph end mark. Through > at least the 6.4.7 release. But by the 7.3.6 release the strikethrough and > underline were removed and the overline remained. Don't have 7.0, 7.1 or 7.2 > build handy to test. But a change in there somewhere that didn't get applied > to the overline. > > Needs a reverse bibisect to find that change, then see if the same handling > can be applied to the overline effect. Bibisected this change with linux-64-7.1 repo to 350dbba44ee7703b1da11cc4732a5090ce5efe3d tdf#105967 sw change tracking: fix pilcrow symbol formatting according to the tracked insertion or deletion of the paragraph break. There was no formatting at all or only (often bad) inheritance of the character style (strike out or underline) of the previous character, confusing the user. Note: because the pilcrow symbol is painted by the temporary text portion SwTmpEndPortion, see sw/qa/extras/layout/data/tdf42748.fodt for testing.
Thanks for the bibisect. @László, * broader CS inheritance beyond track changes instances, more to be done?