Summary: | Spacing issue with Malayalam Sign Virama (U+0D4D) in Writer | ||
---|---|---|---|
Product: | LibreOffice | Reporter: | Ramesh Kunnappully <ramesh1> |
Component: | Writer | Assignee: | خالد حسني <khaled> |
Status: | VERIFIED FIXED | ||
Severity: | normal | CC: | aron.budea, khaled, xiscofauli |
Priority: | medium | Keywords: | bibisected, bisected, regression |
Version: | 7.4.0.3 release | ||
Hardware: | All | ||
OS: | All | ||
See Also: |
https://bugs.documentfoundation.org/show_bug.cgi?id=149017 https://bugs.documentfoundation.org/show_bug.cgi?id=151350 |
||
Whiteboard: | target:7.5.0 inReleaseNotes:7.5 | ||
Crash report or crash signature: | Regression By: | ||
Bug Depends on: | |||
Bug Blocks: | 71732 | ||
Attachments: |
Screenshot
Sample File Screenshot from 7.3.5.2 with correct spacing |
Description
Ramesh Kunnappully
2022-08-29 10:50:21 UTC
Created attachment 182071 [details]
Screenshot
Created attachment 182072 [details]
Sample File
What was the previous LibreOffice version you upgraded from? Upgraded from LibreOffice 7.3.4 Created attachment 182076 [details]
Screenshot from 7.3.5.2 with correct spacing
I get correct spacing with 7.3.5.2 as well.
Khaled Hosny committed a patch related to this issue. It has been pushed to "master": https://git.libreoffice.org/core/commit/29de4d6e1afcf2ad79056e4ad4403c3d4bdda4a0 tdf#150665: Fix justifying spacing marks It will be available in 7.5.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. For the record, this is a regression from author Mark Hung <marklm9@gmail.com> 2022-05-13 00:06:49 +0800 committer Mark Hung <marklh9@gmail.com> 2022-05-18 14:32:06 +0200 commit f8560e09006cec5cc6ef26ccbf4f21aa28c22ac3 (patch) tree 7efdff3730ba7f3797d2770b3da5f57baea2bd61 parent 343eed477ee7b55aa450bbc2aee6786dc0d9a071 (diff) tdf#149017 fix space distribution in SwFntObj::DrawText() Hi Khaled, I found that your patch introduced a new regression in the same document. I'll report it in a follow-up ticket Fix verified in: Version: 7.5.0.0.alpha1+ (X86_64) / LibreOffice Community Build ID: da3dd48eaf9086f8ab28d6a6655f9a638e51433a CPU threads: 8; OS: Linux 5.15; UI render: default; VCL: gtk3 Locale: en-AU (en_AU.UTF-8); UI: en-US Calc: threaded I assume a cherry-pick wasn't done for 7.4 because of the new regression? (In reply to Stéphane Guillou (stragu) from comment #9) > Fix verified in: > > Version: 7.5.0.0.alpha1+ (X86_64) / LibreOffice Community > Build ID: da3dd48eaf9086f8ab28d6a6655f9a638e51433a > CPU threads: 8; OS: Linux 5.15; UI render: default; VCL: gtk3 > Locale: en-AU (en_AU.UTF-8); UI: en-US > Calc: threaded > > I assume a cherry-pick wasn't done for 7.4 because of the new regression? The involved code was changed quite a bit in 7.5 and I don’t think cherry-picking this particular commit alone would be enough. |