Bug 105991 - Changing legend's font changes also the legend position
Summary: Changing legend's font changes also the legend position
Status: NEW
Alias: None
Product: LibreOffice
Classification: Unclassified
Component: Chart (show other bugs)
(earliest affected)
4.3 all versions
Hardware: All All
: medium normal
Assignee: Not Assigned
QA Contact:
Keywords: bibisected, bisected, difficultyBeginner, easyHack, regression, skillCpp
Depends on:
Reported: 2017-02-13 22:20 UTC by Tamás Zolnai
Modified: 2017-08-26 12:13 UTC (History)
4 users (show)

See Also:
Crash report or crash signature:

Test document with a chart (29.30 KB, application/vnd.oasis.opendocument.spreadsheet)
2017-02-13 22:21 UTC, Tamás Zolnai

Note You need to log in before you can comment on or make changes to this bug.
Description Tamás Zolnai 2017-02-13 22:20:36 UTC
When I open the test document and try to change the legend font properties (in chart edit mode) the legend position is also changes when closing Legend dialog (legend is repositioned to the left side of the chart area).

Steps to Reproduce:
1. Open attached test document
2. Double click on chart -> enter into edit mode
3. Double click on legend -> Legend dialog is opened
4. Change font type on Font tab page
-> Font type changes, but Legend also moves to the left.

Actual Results:  
Legend position changes when I set a new font type.

Expected Results:
Legend position should not change.

Reproducible: Always

User Profile Reset: No

Additional Info:

User-Agent: Mozilla/5.0 (Windows NT 6.3; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/56.0.2924.87 Safari/537.36
Comment 1 Tamás Zolnai 2017-02-13 22:21:24 UTC
Created attachment 131195 [details]
Test document with a chart
Comment 2 Buovjaga 2017-02-19 17:24:04 UTC

No problem with 3.6.

Arch Linux 64-bit, KDE Plasma 5
Build ID: 7ec0500e20cf273d70c4fbddb4063b8f8295307c
CPU Threads: 8; OS Version: Linux 4.9; UI Render: default; VCL: kde4; 
Locale: fi-FI (fi_FI.UTF-8); Calc: group
Built on February 18th 2016

Arch Linux 64-bit
Version (Build ID: e183d5b)
Comment 3 Xisco Faulí 2017-02-19 17:45:51 UTC
Regression introduced by:

author	Caolán McNamara <caolanm@redhat.com>	2013-11-27 10:31:37 (GMT)
committer	Caolán McNamara <caolanm@redhat.com>	2013-11-27 10:33:25 (GMT)
commit	07a58db41301d15523519b99d2c64f1885909cda (patch)
tree	1eb5286d2703541b65d1ee144eb300aa52338e59
parent	4f94f16ba15218e5e7a9eb4d72ddb4cb62884dbb (diff)
provide two LegendPositionResources for the interim
LegendPositionResources for .ui based stuff and a legacy
oldLegendPositionResources which can be phased out when
the last .res based user is gone

Adding Cc: to Caolán McNamara
Comment 4 Tamás Zolnai 2017-08-26 12:13:44 UTC
Let's turn this into a easy hack. The code is here:
chart2/uiconfig/ui/tp_LegendPosition.ui -> defines the structure of the font page
chart2/source/controller/dialogs/res_LegendPosition.cxx -> implementation of the font page