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

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
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.

None yet

2 participants