Skip to content

ShellClients: Don't make centered clients docks but allow their shortcut inhibition without dialog#1995

Merged
danirabbit merged 6 commits intomainfrom
leolost/dont-make-centered-clients-docks
Aug 8, 2024
Merged

ShellClients: Don't make centered clients docks but allow their shortcut inhibition without dialog#1995
danirabbit merged 6 commits intomainfrom
leolost/dont-make-centered-clients-docks

Conversation

@leolost2605
Copy link
Copy Markdown
Member

Fixes #1994

We don't really need the focus method now anymore, should we remove it too? Or might it come in handy at some other point in the future?

@leolost2605 leolost2605 requested a review from a team July 20, 2024 14:17
Comment thread src/Dialogs.vala

// Naive check to always allow inhibiting by our settings app. This is needed for setting custom shortcuts
if (app.id == "io.elementary.settings.desktop") {
if (app.id == "io.elementary.settings.desktop" || // Naive check to always allow inhibiting by our settings app. This is needed for setting custom shortcuts
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

While we're here should we just do prefix of io.elementary?

Comment thread src/Dialogs.vala
Comment thread src/ShellClients/ShellClientsManager.vala Outdated
@danirabbit danirabbit requested a review from tintou July 22, 2024 17:40
@leolost2605 leolost2605 mentioned this pull request Aug 8, 2024
Copy link
Copy Markdown
Member

@tintou tintou left a comment

Choose a reason for hiding this comment

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

Great!

@danirabbit danirabbit merged commit 31b8939 into main Aug 8, 2024
@danirabbit danirabbit deleted the leolost/dont-make-centered-clients-docks branch August 8, 2024 20:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

No open projects
Status: Done

Development

Successfully merging this pull request may close these issues.

Redesign center windows

3 participants