After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 788898 - Inspector: Default paned split position/ratio between tree and properties is not very useful
Inspector: Default paned split position/ratio between tree and properties is ...
Status: RESOLVED FIXED
Product: gtk+
Classification: Platform
Component: Debugging
3.22.x
Other All
: Normal normal
: ---
Assigned To: gtk-bugs
gtk-bugs
Depends on:
Blocks:
 
 
Reported: 2017-10-12 19:55 UTC by Daniel Boles
Modified: 2017-11-01 20:25 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
screenshot of Inspector initially, with separator at 1/3 from left (80.15 KB, image/png)
2017-10-12 19:55 UTC, Daniel Boles
  Details
screenshot of Inspector with suggested position of separator at halfway mark (95.88 KB, image/png)
2017-10-12 19:56 UTC, Daniel Boles
  Details
screenshot with default paned separator position at 400px of window's 800px (74.84 KB, image/png)
2017-10-12 20:09 UTC, Daniel Boles
  Details
Inspector: better default split of CSS nodes/props (1.27 KB, patch)
2017-10-12 20:12 UTC, Daniel Boles
committed Details | Review

Description Daniel Boles 2017-10-12 19:55:24 UTC
Created attachment 361451 [details]
screenshot of Inspector initially, with separator at 1/3 from left

The CSS Nodes page initially has the paned separator about 1/3rd from the left, which in makes the CSS pane too narrow and means its State column can't be seen until the paned is resized. This doesn't seem justified as I doubt the properties pane often needs to be that wide.

50:50 seems like a better default split point.
Comment 1 Daniel Boles 2017-10-12 19:56:10 UTC
Created attachment 361452 [details]
screenshot of Inspector with suggested position of separator at halfway mark
Comment 2 Daniel Boles 2017-10-12 20:09:21 UTC
Created attachment 361454 [details]
screenshot with default paned separator position at 400px of window's 800px

window.ui sets Window:default-size to 800px wide, so we probably don't want to go above 400px for the paned separator position, even if on a system like mine the window ends up being wider anyway.

Here's what that looks like
Comment 3 Daniel Boles 2017-10-12 20:12:56 UTC
Created attachment 361455 [details] [review]
Inspector: better default split of CSS nodes/props

Move the default pos of the Paned handle to 400px from the left, i.e.
50% of the default width of the window. The previous position at 300px
from left meant the node treeview was too narrow & could easily result
in the (useful) State column not being visible in the case of many
apps. The properties pane doesn't need to be as big as it was anyway.
Comment 4 Daniel Boles 2017-11-01 20:25:54 UTC
Attachment 361455 [details] pushed as 55210ae - Inspector: better default split of CSS nodes/props