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

Transient secondary windows #98

Merged
merged 1 commit into from
Oct 6, 2022
Merged

Transient secondary windows #98

merged 1 commit into from
Oct 6, 2022

Conversation

Schmiddiii
Copy link
Collaborator

@Schmiddiii Schmiddiii commented Oct 5, 2022

Licensing

  • I confirm that this is either my code or was released under the terms of a GPLv3-or-later compatible license. Also I agree to release it here under the terms of the GPLv3-or-later.

Description

Transient About- and Preferences-Window. The import-window was already transient.

Ping @TheEvilSkeleton for testing.

Linked Issue

Related to #95.

@TheEvilSkeleton
Copy link
Contributor

I can confirm that both Preferences and About windows are now transient. However, the Import window is not.

2022-10-05.16-31-22.mp4

@Schmiddiii
Copy link
Collaborator Author

Maybe now?

To be honest I do not even know what transient-for is even doing, but in the GTK-inspector the property is set to the main window and I do not get a warning that dialogs without transient parent is discouraged.

@TheEvilSkeleton
Copy link
Contributor

transient-for basically sticks the child window with the parent window, so they're not disconnected.

Here's a video that showcases the differences. When I open Preferences and I move the child window around, the parent window follows it as well. However, when I open the Import window, it moves separately.

2022-10-06.11-26-36.mp4

Copy link
Contributor

@TheEvilSkeleton TheEvilSkeleton left a comment

Choose a reason for hiding this comment

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

I just tried it and the Import window is still not transient, from the looks of it. I guess it isn't a big deal since we're going to redesign the window and make sure it works well.

@Schmiddiii
Copy link
Collaborator Author

Then I am just going to merge it for now and work on porting the import window to AdwMessageDialog tomorrow.

@Schmiddiii Schmiddiii merged commit 72c62c5 into master Oct 6, 2022
@Schmiddiii Schmiddiii deleted the transient-windows branch October 6, 2022 15:58
# for free to subscribe to this conversation on GitHub. Already have an account? #.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants