Skip to content
This repository has been archived by the owner on Mar 6, 2024. It is now read-only.

Use YaruDialogTitleBar #1323

Merged
merged 1 commit into from Jan 20, 2023
Merged

Use YaruDialogTitleBar #1323

merged 1 commit into from Jan 20, 2023

Conversation

jpnurmi
Copy link
Contributor

@jpnurmi jpnurmi commented Jan 19, 2023

This not only makes the dialogs look nicer but also adds such behavior that could be considered a pre-requisite for replacing the native HdyHeaderBar with YaruWindowTitleBar (outside vs. inside the Flutter view, respectively).

When a modal dialog is open, the entire Flutter view is covered by a modal barrier. This means that YaruWindowTitleBar is not accessible underneath whenever a modal dialog is open. YaruDialogTitleBar solves this by offering the usual top-level window actions (move by drag, right-click menu) via the dialog title bar. It feels a lot like a modal top-level dialog attached to and blocking its parent window.

Before After
image image
image image
image image
image image
image image

Ref: #670

@jpnurmi

This comment was marked as outdated.

Copy link
Member

@d-loose d-loose left a comment

Choose a reason for hiding this comment

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

LGTM 👍

@jpnurmi jpnurmi merged commit 84b194c into canonical:main Jan 20, 2023
@jpnurmi jpnurmi deleted the dialog-title-bar branch January 20, 2023 16:55
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants