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 783920 - panels are not restored to their proper state
panels are not restored to their proper state
Status: RESOLVED FIXED
Product: gnome-builder
Classification: Other
Component: libide
unspecified
Other Linux
: Normal normal
: ---
Assigned To: GNOME Builder Maintainers
GNOME Builder Maintainers
Depends on:
Blocks:
 
 
Reported: 2017-06-18 10:40 UTC by Christian Hergert
Modified: 2017-06-18 10:41 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
perspective: add hooks to restore state (2.12 KB, patch)
2017-06-18 10:40 UTC, Christian Hergert
committed Details | Review
editor: implement restore_state (3.86 KB, patch)
2017-06-18 10:40 UTC, Christian Hergert
committed Details | Review
workbench: use restore_state api from perspectives (3.36 KB, patch)
2017-06-18 10:40 UTC, Christian Hergert
committed Details | Review

Description Christian Hergert 2017-06-18 10:40:08 UTC
We store panel state for the editor panels in (org.gnome.builder.workbench, which as an aside, could be in a better place) and that is not being restored correctly when a project is loaded.

They are always being displayed even when previously hidden.

Patches forthcoming.
Comment 1 Christian Hergert 2017-06-18 10:40:27 UTC
Created attachment 353976 [details] [review]
perspective: add hooks to restore state

These should be used to restore state like panels. It is called after the
perspectives have been added and it should be relatively stable to cleanly
restore settings state. By stable, I mean widgets aren't still being added.
Comment 2 Christian Hergert 2017-06-18 10:40:31 UTC
Created attachment 353977 [details] [review]
editor: implement restore_state

This uses the new restore_state vfunc, and ensures that we do not animate
the panels while restoring them.
Comment 3 Christian Hergert 2017-06-18 10:40:34 UTC
Created attachment 353978 [details] [review]
workbench: use restore_state api from perspectives

This allows the perspectives to restore state after the common work of
loading extensions should have occurred. This fixes an issue where our
panels were not being shown/hidden at startup properly.
Comment 4 Christian Hergert 2017-06-18 10:41:38 UTC
Attachment 353976 [details] pushed as 6402956 - perspective: add hooks to restore state
Attachment 353977 [details] pushed as 2c993be - editor: implement restore_state
Attachment 353978 [details] pushed as dbe559e - workbench: use restore_state api from perspectives