Skip to content

multiple panels: make sure exactly one title button set on left/start is visible regardless of leaflet state

Currently, Settings is inconsistent with displaying the left title buttons with having anything between zero and two.

Methodology:

  1. I comprised a list of every AdwHeaderBar in a .ui file.
  2. I made sure every panel can be displayed in a flatpak build based on !1693 (closed) with a workaround for the mouse panel that failed to init the test dialog and crashed subsequently: cd3482d5
  3. I checked that the dialog windows have the same value for show-start- and -end-title-buttons. They all did.
  4. For the panels, I navigated to each one and noted missing or duplicate title buttons and noted a missing back button in the mouse panel.

After applying my changes, I repeated steps 3 and 4 to verify the behaviour is now as expected.

To make sure one left title button and/or one right title button are always visible simultaneously, I tested with close:close:

folded before folded after expanded before expanded after
Application overview image image
Application image image image image
Display overview image image image image
Display image image image image
Night Light image image image image
Mouse image image image image
Network image image image image
Accessibility image image image image
Accessibility / Seeing image image image image
User image image image image

Not fixed: The title buttons are hidden for the apply header bar of the displays panel, but not the start title buttons of the sidebar. This, however, is only noticeable, if both start and end title buttons are used and not just either one or the other. (Note that the button layout still is close:close for the following screenshots.)

folded expanded
image image
Edited by Markus Göllnitz

Merge request reports