Created attachment 147571 [details] screenshot and file Open the workbook in Calc Double-click the chart Select Data series Occurrences from Select chart element Click Format Selection (next to Select chart element) Issue: Hide Legend Entry and its checkbox are not aligned vertically, and the space between is inconsistently large compared with the other checkbox. Version: 6.3.0.0.alpha0+ (x64) Build ID: 3c964980da07892a02d5ac721d80558c459532d0 CPU threads: 1; OS: Windows 10.0; UI render: default; VCL: win; TinderBox: Win-x86_64@42, Branch:master, Time: 2018-12-12_03:38:09 Locale: en-US (en_US); UI-Language: en-US Calc: threaded
Reproduced for Version: 6.3.0.0.alpha0+ Build ID: e3e15d8c625cbf56e1f03d9ebde966e78844c7a4 CPU threads: 1; OS: Linux 4.14; UI render: default; VCL: kde5; TinderBox: Linux-rpm_deb-x86_64@86-TDF, Branch:master, Time: 2018-12-13_20:10:25 Locale: ru-RU (ru_RU.UTF-8); UI-Language: en-US Calc: threaded
This is a new feature from https://cgit.freedesktop.org/libreoffice/core/commit/?id=19b2ed4dee1ec33ad55473d43bfcd0bfa194cbee @Heiko, do you feel like fixing it?
Adolfo Jayme Barrientos committed a patch related to this issue. It has been pushed to "master": https://git.libreoffice.org/core/+/9df7e3706b357114d4d02bb9a8b4261341faf539%5E%21 tdf#122121 Fix checkbox alignment & spacing in Data Series dialog It will be available in 6.3.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.
Created attachment 147662 [details] how it looks after the commit
Verified in Version: 6.3.0.0.alpha0+ Build ID: 78073ecfdc50e78e3ce094c1259779b7c3b88bc4 CPU threads: 4; OS: Linux 4.15; UI render: default; VCL: gtk3; Locale: ca-ES (ca_ES.UTF-8); UI-Language: en-US Calc: threaded @Adolfo Jayme, Thanks for the quick fix!!
@Adolfo Jayme, btw, you should not push your patches directly to master... use gerrit...
Xisco: 1) A patch as TRIVIAL as the one I pushed above will not break CI. I’m not wasting electricity and hardware resources for a useless CI validation. I’m sure it works because I tested it before pushing. 2) AFAIK, skipping Gerrit reviewing is allowed in the master branch. If the commit rules have changed, can you point me to the corresponding wiki doc change?