Bug 154731 - Enhancement: Simplify Zoom & View Layout dialog by eliminating unusable choices
Summary: Enhancement: Simplify Zoom & View Layout dialog by eliminating unusable choices
Status: RESOLVED FIXED
Alias: None
Product: LibreOffice
Classification: Unclassified
Component: Draw (show other bugs)
Version:
(earliest affected)
7.4.6.2 release
Hardware: All All
: medium enhancement
Assignee: Baole Fang
URL:
Whiteboard: target:7.6.0
Keywords: difficultyBeginner, easyHack, skillCpp, topicUI
Depends on:
Blocks: Zoom
  Show dependency treegraph
 
Reported: 2023-04-09 16:29 UTC by Don Matschull
Modified: 2023-04-21 07:14 UTC (History)
3 users (show)

See Also:
Crash report or crash signature:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Don Matschull 2023-04-09 16:29:35 UTC
The right side of the Zoom & View Layout dialog has an area titled "View Layout". This area was grayed out when I first saw it, so I searched to find out when and how it might be used.

I finally found a reference to the Zoom & View Layout in Chapter 3 of the Draw Guide that says of the View Layout area: "Used to set the view layout and is not available for drawings. View layout is normally used in text documents to see the effects of different layout settings."

Since this area is "not available for drawings", removing it from the dialog would eliminate confusion. The less confusing an application is, the easier it is to use.
Comment 1 V Stuart Foote 2023-04-09 16:55:24 UTC
The options are greyed out/non-selectable and that is sufficient as hiding it from the global Zoom & View layout would require either splitting the two functions, or unreasonable rework of the dialog.

IMHO => WF
Comment 2 Heiko Tietze 2023-04-11 09:24:41 UTC
Should be easyhackable to hide the controls instead of making it disabled.

The UI is cui/uiconfig/ui/zoomdialog.ui loaded in cui/source/dialogs/zoom.cxx and the function in question m_xViewFrame->set_sensitive(false);
Comment 3 Baole Fang 2023-04-17 19:08:43 UTC
Start working on this
Comment 4 Baole Fang 2023-04-20 17:36:03 UTC
I've submitted a patch here. Can anyone review it?

https://gerrit.libreoffice.org/c/core/+/150667
Comment 5 Commit Notification 2023-04-21 07:14:42 UTC
Baole Fang committed a patch related to this issue.
It has been pushed to "master":

https://git.libreoffice.org/core/commit/56d2939e1928ae06fa74cb4b4be36cc0eaf3e702

tdf#154731: Hide Zoom & View Layout dialog

It will be available in 7.6.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.