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 783429 - 3.26 redesign: subpanel pages aren't automatically focused
3.26 redesign: subpanel pages aren't automatically focused
Status: RESOLVED FIXED
Product: gnome-control-center
Classification: Core
Component: general
unspecified
Other Linux
: Normal normal
: ---
Assigned To: Control-Center Maintainers
Control-Center Maintainers
Depends on:
Blocks:
 
 
Reported: 2017-06-05 13:27 UTC by Jeremy Bicha
Modified: 2017-07-25 01:26 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
shell/alt: Always activate panel list after view change (1.91 KB, patch)
2017-07-14 17:08 UTC, Florian Müllner
none Details | Review
shell/alt: Always activate panel list after view change (3.49 KB, patch)
2017-07-25 01:23 UTC, Georges Basile Stavracas Neto
committed Details | Review

Description Jeremy Bicha 2017-06-05 13:27:08 UTC
gnome-control-center master built in jhbuild on Ubuntu GNOME 17.04

Details and Devices are the two section of gnome-control-center-alt that have subpages.

When I click either Details or Devices, I expect the subpanel page to show up. What currently happens is that a page is selected on the left but the right content area still shows whatever older page was selected before clicking Details or Devices.
Comment 1 Florian Müllner 2017-07-14 17:08:22 UTC
Created attachment 355617 [details] [review]
shell/alt: Always activate panel list after view change

The displayed panel should always match the selection in the panel
list. That implies that after swapping out the list (for instance
by browsing Devices or Details), we need to update the panel rather
than keeping the old one around until the user manually selects a
row. The easiest way to achieve that is to activate the panel list
after a view change when appropriate.
Comment 2 Georges Basile Stavracas Neto 2017-07-25 01:00:09 UTC
Review of attachment 355617 [details] [review]:

This patch causes a crash when opening a panel from the command line. E.g." $ shell/gnome-control-center-alt keyboard"
Comment 3 Georges Basile Stavracas Neto 2017-07-25 01:23:46 UTC
Created attachment 356328 [details] [review]
shell/alt: Always activate panel list after view change

The displayed panel should always match the selection in the panel
list. That implies that after swapping out the list (for instance
by browsing Devices or Details), we need to update the panel rather
than keeping the old one around until the user manually selects a
row. The easiest way to achieve that is to activate the panel list
after a view change when appropriate.
Comment 4 Georges Basile Stavracas Neto 2017-07-25 01:24:57 UTC
Comment on attachment 356328 [details] [review]
shell/alt: Always activate panel list after view change

Fixed the crash by keeping track of when we want to autoselect the first panel or not. When setting programatically, don't autoselect; otherwise, do it.
Comment 5 Georges Basile Stavracas Neto 2017-07-25 01:26:16 UTC
Thanks for the patch.

Attachment 356328 [details] pushed as 8b09079 - shell/alt: Always activate panel list after view change