Bug 142514 - Inconsistent use and non-antialiased graphical representation and of rotation wheels
Summary: Inconsistent use and non-antialiased graphical representation and of rotatio...
Status: UNCONFIRMED
Alias: None
Product: LibreOffice
Classification: Unclassified
Component: UI (show other bugs)
Version:
(earliest affected)
7.1.3.1 rc
Hardware: All All
: medium normal
Assignee: Not Assigned
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2021-05-26 21:13 UTC by Thorsten Wagner
Modified: 2021-06-03 13:19 UTC (History)
1 user (show)

See Also:
Crash report or crash signature:


Attachments
Screenshot 1 (84.04 KB, image/png)
2021-05-26 21:18 UTC, Thorsten Wagner
Details
Screenshot 2 (83.17 KB, image/png)
2021-05-26 21:18 UTC, Thorsten Wagner
Details
Screenshot 3 (115.05 KB, image/png)
2021-05-26 21:18 UTC, Thorsten Wagner
Details
Screenshot 4 (76.04 KB, image/png)
2021-05-26 21:19 UTC, Thorsten Wagner
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Thorsten Wagner 2021-05-26 21:13:15 UTC
Description:
Rotation wheels are used in Impress/Draw (see screenshots 1 and 2) within sidebar and Position and Size dialog. In Calc rotation wheel is used in Format Cells dialog only (see screenshot 3), but not within sidebar (see screenshot 4).

All graphics representations are not antialiased on macOS and with GTK at least. Furthermore there are dark grey areas behind rotation wheels within dialogs. They do not seem to be intended.

As all actions are possible with simple text fields and steppers, I suggest dropping rotation wheels completely (from Calc and Impress/Draw and from sidebar and dialogs as well). This would simplify UI and reduce development effort.

Steps to Reproduce:
1. Open Calc and Impress/Draw
2. Look at the sidebars and corresponding dialogs

Actual Results:
Rotation wheels are shown non-antialiased and with grey background within dialogs

Expected Results:
There are no rotation wheels


Reproducible: Always


User Profile Reset: No



Additional Info:
Version: 7.1.3.2.0+ / LibreOffice Community
Build ID: 162213c78ede269b255c1eddaaac013975a34ee2
CPU threads: 8; OS: Mac OS X 10.15.7; UI render: GL; VCL: osx
Locale: de-DE (en_US.UTF-8); UI: en-US
Calc: threaded
Comment 1 Thorsten Wagner 2021-05-26 21:18:19 UTC
Created attachment 172380 [details]
Screenshot 1
Comment 2 Thorsten Wagner 2021-05-26 21:18:35 UTC
Created attachment 172381 [details]
Screenshot 2
Comment 3 Thorsten Wagner 2021-05-26 21:18:53 UTC
Created attachment 172382 [details]
Screenshot 3
Comment 4 Thorsten Wagner 2021-05-26 21:19:12 UTC
Created attachment 172383 [details]
Screenshot 4
Comment 5 Heiko Tietze 2021-05-27 08:48:45 UTC
While I wont shed a tear over dropping the dialcontrol [1] (besides antialiasing, transparency it is not a11y conform and takes a lot of screen space), we need some kind of preview. Take the rotation angle of shapes as an example (property dialog): What exactly does -45° mean? Wouldn't it be easier to fix the issues?

[1] svx/source/dialog/dialcontrol.cxx
Comment 6 Thorsten Wagner 2021-05-27 12:25:32 UTC
Changing rotation by means of sidebar has an immediate visual feedback. Rotation wheel within sidebar is not required in Impress/Draw from this point of view. Calc has no rotation wheel in sidebar already.

Concerning dialogs a preview is nice (if visual represenation is nice), but optional from my point of view too:

(1) Counting increasing angles as mathematically positive (or couterclockwise) is standard behaviour for most applications.

(2) Looking at the result is only a few clicks away (closing dialog and reopening dialog if direction was wrong)

(3) To make direction clear a text label is maybe enough for description.
Comment 7 Heiko Tietze 2021-05-27 13:08:42 UTC
> git grep dialcontrol.hxx
chart2/source/controller/dialogs/res_DataLabel.hxx
chart2/source/controller/dialogs/tp_AxisLabel.hxx
chart2/source/controller/dialogs/tp_PolarOptions.hxx
chart2/source/controller/dialogs/tp_TitleRotation.hxx
cui/source/inc/align.hxx
cui/source/inc/transfrm.hxx
svx/source/sidebar/possize/PosSizePropertyPanel.cxx
sw/source/uibase/inc/frmpage.hxx


+1 for the sidebar but -0.2 for the chart dialog, alignment and transformation. More opinions please.
Comment 8 Heiko Tietze 2021-06-03 12:51:07 UTC
No further input - had this topic on the agenda for the design meeting and mentioned it on ESC. Keeping the ticket and removing the keyword.
Comment 9 Thorsten Wagner 2021-06-03 13:19:34 UTC
Looked at the code in the meantime: Removal from sidebar is an easy change, antialiasing within dialogs is a general issue but resulting poor quality is noticeable for the rotation wheels especially. Besides rotation wheels there is no pressure to fix antialiasing at the moment.