Skip to content

Conversation

pjungkamp
Copy link
Contributor

See #1694 (comment)

Did you think of something like this? @swsnr @sdroege

Copy link
Member

@sdroege sdroege left a comment

Choose a reason for hiding this comment

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

@swsnr If that also matches what you thought of, let's merge it :)

Copy link
Contributor

@swsnr swsnr left a comment

Choose a reason for hiding this comment

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

Looks good to me 👍

@sdroege sdroege marked this pull request as ready for review August 8, 2025 11:50
@sdroege sdroege merged commit 6f3e317 into gtk-rs:main Aug 8, 2025
46 of 48 checks passed
@pjungkamp
Copy link
Contributor Author

That was only a first draft. This isn't remotely working yet. The new module isn't even included. Half the imports aren't used and the last line is implementing ApplicationExtManual for Application, not ApplicationCommandLineExtManual for ApplicationCommandLine.

@sdroege
Copy link
Member

sdroege commented Aug 8, 2025

Oh no, I assumed this was ready apart from getting feedback. Sorry for merging it already.

@swsnr
Copy link
Contributor

swsnr commented Aug 8, 2025

Sorry from my side as well. I noticed that this wasn't complete but assumed I was only supposed to comment on the general idea 😔

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.

3 participants