| Summary: | EDITING Pie chart labels “Best fit” placement overlaps other chart slices | ||
|---|---|---|---|
| Product: | LibreOffice | Reporter: | NISZ LibreOffice Team <libreoffice> |
| Component: | Chart | Assignee: | Balázs Varga <varga.balazs3> |
| Status: | VERIFIED FIXED | ||
| Severity: | normal | CC: | buzea.bogdan, ilmari.lauhakangas, nemeth, varga.balazs3 |
| Priority: | medium | ||
| Version: | 7.0.0.0.alpha1+ | ||
| Hardware: | All | ||
| OS: | All | ||
| See Also: | https://bugs.documentfoundation.org/show_bug.cgi?id=135810 | ||
| Whiteboard: | target:7.1.0 target:7.0.0.1 | ||
| Crash report or crash signature: | Regression By: | ||
| Bug Depends on: | |||
| Bug Blocks: | 90486 | ||
| Attachments: |
Example file from Excel
Screenshot of the original document side by side in Excel and Calc this is a screenshot from 7.1 |
||
|
Description
NISZ LibreOffice Team
2020-05-25 14:40:18 UTC
Created attachment 161265 [details]
Screenshot of the original document side by side in Excel and Calc
Confirm it. Version: 6.4.4.2 Build ID: 3d775be2011f3886db32dfd395a6a6d1ca2630ff CPU threads: 4; OS: Linux 5.4; UI render: default; VCL: gtk3; Locale: ro-RO (ro_RO.UTF-8); UI-Language: en-US Calc: threaded Balazs Varga committed a patch related to this issue. It has been pushed to "master": https://git.libreoffice.org/core/commit/73477348e30c6931a537cba5557c250183fbeb9b tdf#133376 Chart view: improve BestFit position of data labels It will be available in 7.1.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. Balazs Varga committed a patch related to this issue. It has been pushed to "libreoffice-7-0": https://git.libreoffice.org/core/commit/263c3b56fc36cd9435b57bfde8ff9b38a7d894b7 tdf#133376 Chart view: improve BestFit position of data labels It will be available in 7.0.0.1. 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. Created attachment 161688 [details]
this is a screenshot from 7.1
It's not the same like in Excel, but at least is much better center.
See screenshot.
The test added with the fix is unreliable. It fails, if using font scaling on Windows. Similar situations have been fixed like this: https://gerrit.libreoffice.org/c/core/+/85967 https://gerrit.libreoffice.org/c/core/+/85974 https://gerrit.libreoffice.org/c/core/+/75302 https://gerrit.libreoffice.org/c/core/+/65595 A "brute force" fix example (skipping the test): https://gerrit.libreoffice.org/c/core/+/65595 Relevant patch by Mike K.: https://gerrit.libreoffice.org/c/core/+/106661 Introduce BootstrapFixture::IsDefaultDPI convenience function |