Bug 112615

Summary: Customization dialog is too wide
Product: LibreOffice Reporter: Yousuf Philips (jay) (retired) <philipz85>
Component: LibreOfficeAssignee: Not Assigned <libreoffice-bugs>
Status: RESOLVED WONTFIX    
Severity: normal CC: heiko.tietze, kara, xiscofauli
Priority: medium Keywords: bibisected, bisected, regression
Version: 6.0.0.0.alpha0+   
Hardware: All   
OS: All   
Whiteboard:
Crash report or crash signature: Regression By:
Bug Depends on:    
Bug Blocks: 103238    
Attachments: red 100px areas in both function lists

Description Yousuf Philips (jay) (retired) 2017-09-24 13:29:48 UTC
Created attachment 136511 [details]
red 100px areas in both function lists

In 5.4, the customization dialog was 605x671, while the new version in 6.0 is 874x673, a width increase of 269px, when it could easily be reduced by 200+ pixels by reducing the minimum widths of the 2 function lists, which will reduce the large amount of white space in the dialog.
Comment 1 Xisco FaulĂ­ 2017-10-10 14:31:53 UTC
Regression introduced by:

author	Muhammet Kara <muhammet.kara@pardus.org.tr>	2017-09-11 12:42:30 (GMT)
committer	Heiko Tietze <tietze.heiko@googlemail.com>	2017-09-12 19:48:20 (GMT)
commit e970395c692a5c315914ddf5b43cf01e590345ff (patch)
tree 7aa71acfbdbd6fd80b079d498d8f20374d29251b
parent 9659efe3f695aeeffc4cae427b80020fd2d96796 (diff)
tdf#112241: Improve column width behaviour in Customize dialog
Also:
 * Remove unnecessary frame "contents"
 * Remove unnecessary alignment
 * Remove extra column from the left grid (grid4)
 * Move all into a new grid for better sizing

Bisected with: bibisect-linux64-6.0

Adding Cc: to Muhammet Kara
Comment 2 Heiko Tietze 2017-10-15 07:07:48 UTC
Talked to Muhammet and the thing is that the dialog width is being calculated from the text in the list. Obviously not the best approach and he will turn this into a normal Glade accessible ui property.
Comment 3 Muhammet Kara 2018-10-08 14:04:28 UTC
It can be reduced to 800px easily. Would that be enough?
Comment 4 Muhammet Kara 2018-10-08 14:06:30 UTC
Forgot to mention how. Sorry for the noise.

Here is the relevant line: https://opengrok.libreoffice.org/xref/core/cui/source/customize/cfg.cxx#1152

One can achieve 800px width just by reducing the first number.
Comment 5 Heiko Tietze 2018-10-08 15:07:33 UTC
These few pixels are not visible unless you compare directly. And actually I disagree with the need to cut off all whitespace. The dialog is well designed and provides room for the various string length. Since no body agreed with Jay I close the ticket as WF.