Skip to content

Conversation

@danirabbit
Copy link
Member

Some proposed changes to the popover

  • Use MENU style. Saves us a bunch of manually setting of margins, padding etc
  • Remove the icon from the header and use Granite.HeaderLabel. More clearly visually separates the header from the list of apps stylistically
  • Make the secondary text of the header a little more concise
  • Set the header as mnemonic widget for the list so the screenreader associates it and reads it correctly
  • Make secondary text for app rows small text to give more visual hierarchy and make more room for long labels
  • Change tooltip for button to "Quit" to be more concise
  • Make buttons destructive styled and slightly smaller

AFTER

Screenshot from 2025-07-07 14 43 53

BEFORE

Screenshot from 2025-07-07 14 49 29

@danirabbit danirabbit requested a review from leolost2605 July 7, 2025 21:53
Copy link
Member

@leolost2605 leolost2605 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Much better, thank!

@leolost2605 leolost2605 merged commit af131bd into leolost/background-apps Jul 7, 2025
4 checks passed
@leolost2605 leolost2605 deleted the danirabbit/background-apps-popover branch July 7, 2025 22:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants