Skip to content

Port to GTK4 - #29

Draft
danirabbit wants to merge 8 commits into
mainfrom
danirabbit/gtk4
Draft

Port to GTK4#29
danirabbit wants to merge 8 commits into
mainfrom
danirabbit/gtk4

Conversation

@danirabbit

Copy link
Copy Markdown
Member

Fixes #8

@danirabbit danirabbit moved this to In progress in OS 8.1.0 Jun 30, 2025
@danirabbit danirabbit removed this from OS 8.1.0 Oct 3, 2025
@danirabbit danirabbit moved this to In Progress in GTK4 Porting Apr 15, 2026
@zeebok

zeebok commented Apr 27, 2026

Copy link
Copy Markdown

Are there any other PRs relating to this that need to be looked at before this is ready for review?

@danirabbit

Copy link
Copy Markdown
Member Author

@zeebok i don't think there are any more prep PRs. I just need to set aside some time to finish it :)

delete_event.connect (() => {
close_request.connect (() => {
if (transfer.status == "active") {
return hide_on_delete ();

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.

Suggested change
return hide_on_delete ();
hide ();
return true;

I haven't tested yet, but reading valadoc it sounds like doing the same thing hide_on_delete () used to do in GTK 3 would work here?

The function calls hide on its argument, then returns true.

delete_event.connect (() => {
close_request.connect (() => {
if (transfer.status == "active") {
return hide_on_delete ();

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.

Suggested change
return hide_on_delete ();
hide ();
return true;

Same here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: In Progress

Development

Successfully merging this pull request may close these issues.

Port to GTK 4

3 participants