Bug 160276 - Writer comment: Author and Date info should use same padding as is used for comment text
Summary: Writer comment: Author and Date info should use same padding as is used for c...
Status: VERIFIED FIXED
Alias: None
Product: LibreOffice
Classification: Unclassified
Component: Writer (show other bugs)
Version:
(earliest affected)
24.8.0.0 alpha0+
Hardware: All All
: low enhancement
Assignee: i.dot
URL:
Whiteboard: target:26.2.0
Keywords: difficultyBeginner, easyHack, skillCpp, topicDesign
Depends on:
Blocks: Writer-Comments
  Show dependency treegraph
 
Reported: 2024-03-19 14:04 UTC by steve
Modified: 2025-09-24 07:45 UTC (History)
3 users (show)

See Also:
Crash report or crash signature:


Attachments
Screenshot 2024-03-19 Author and Date should use same padding as comment text (256.38 KB, image/png)
2024-03-19 14:05 UTC, steve
Details
screenshot with "reply to" header, and "resolved" status (90.33 KB, image/png)
2024-04-04 03:28 UTC, Stéphane Guillou (stragu)
Details
Screenshot 2025-09-23 improved, not yet perfectly aligned (1.32 MB, image/png)
2025-09-23 17:03 UTC, steve
Details

Note You need to log in before you can comment on or make changes to this bug.
Description steve 2024-03-19 14:04:55 UTC
Description:
Writer comment: Author and Date info should use same padding as is used for comment text

Steps to Reproduce:
Create a comment in writer.

Actual Results:
Author and Timestamp are stuck to the left boarder of the comment field.

Expected Results:
Author and Timestamp should use same padding as is used for the comment text.


Reproducible: Always


User Profile Reset: No

Additional Info:
Version: 24.8.0.0.alpha0+ (X86_64) / LibreOffice Community
Build ID: 479b5bbe8ca2177ba7574e7aa2308b5d0de1895c
CPU threads: 8; OS: macOS 13.6.5; UI render: Skia/Metal; VCL: osx
Locale: en-US (en_US.UTF-8); UI: en-US
Calc: threaded
Comment 1 steve 2024-03-19 14:05:12 UTC
Created attachment 193195 [details]
Screenshot 2024-03-19 Author and Date should use same padding as comment text
Comment 2 Stéphane Guillou (stragu) 2024-04-04 03:26:51 UTC
OOo 3.3 and 24.2.2.2 have both flush to the left border.
Padding was added by 1dc0c8c8304ebbce0a2e05e89a3b5458d88f3630 for bug 159531 
(should have looked at the "see also" before bibisecting, but I was mislead by the 3.3 as earliest version affected 😅)

Heiko says in bug 159531 comment 9:
> Different code, probably not easy to find. Having the author information drawn differently to the content is not bad (although spacing is our friend).

Note that the "Reply to" header has the same padding as the comment.

I agree that the spacing of the authorship info could be improved, it feels too close to the edge now that everything else is spaced. And there is no extra space between text and authorship info.

More opinion from other UX/Design contributors?
Comment 3 Stéphane Guillou (stragu) 2024-04-04 03:28:27 UTC
Created attachment 193477 [details]
screenshot with "reply to" header, and "resolved" status
Comment 4 Heiko Tietze 2024-04-04 08:39:16 UTC
No objection to a few pixel whitespace. But as additional info these lines do not need to align with the actual comment and are less important.

Potential code pointer: SwAnnotationWin::DrawForPage() in sw/source/uibase/docvw/AnnotationWin2.cxx
Comment 5 Commit Notification 2025-09-16 14:45:02 UTC
Manas Todi committed a patch related to this issue.
It has been pushed to "master":

https://git.libreoffice.org/core/commit/57b3eb385b4ae100c5121d47845b46084fd9ed19

tdf#160276: CUI: Increase left margin for comment author and date labels

It will be available in 26.2.0.

The patch should be included in the daily builds available at
https://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More
information about daily builds can be found at:
https://wiki.documentfoundation.org/Testing_Daily_Builds

Affected users are encouraged to test the fix and report feedback.
Comment 6 steve 2025-09-23 17:03:31 UTC
Created attachment 202938 [details]
Screenshot 2025-09-23 improved, not yet perfectly aligned

This has improved. Not sure what the expected structure should be. If alginment should be there, this is not it. If the indentation is expected, guess we could call this fixed and Author + Date info no longer is glued to the left side and looking a bit better now.