Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: hide all windows (show desktop) via Super+D #1831

Merged
merged 1 commit into from
Oct 25, 2024

Conversation

salim-b
Copy link
Contributor

@salim-b salim-b commented Oct 25, 2024

Upstream issue proposing this (years ago) is found here: https://gitlab.gnome.org/GNOME/gsettings-desktop-schemas/-/issues/24

Justification to not set the shortcut by default by GNOME dev (Ubuntu has that default) is:

The original purpose of that shortcut was to provide quick access to desktop icons (in fact, the internal shortcut name is show-desktop). As we no longer have those, the shortcut is no longer useful to most users.

Main counter-argument is: Being able to hide all windows (and showing one of Bluefin's beautiful wallpapers 🦕 or whatever) is already a use case.

GNOME dev counters that with

Sure, and nobody is planning to remove the shortcut. But it's not a strong argument for a default shortcut, given that someone who installs and configures conky can also assign a shortcut.

Now I just wanted to see if anyone is against setting this by default in Bluefin.

@dosubot dosubot bot added size:XS This PR changes 0-9 lines, ignoring generated files. enhancement New feature or request labels Oct 25, 2024
@castrojo
Copy link
Member

Bringing the classic Ubuntu desktop is what we do around here, appreciate this one!

@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label Oct 25, 2024
@castrojo castrojo added this pull request to the merge queue Oct 25, 2024
Merged via the queue into ublue-os:main with commit 3c60627 Oct 25, 2024
44 of 46 checks passed
@salim-b salim-b deleted the patch-2 branch October 25, 2024 01:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request lgtm This PR has been approved by a maintainer size:XS This PR changes 0-9 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants