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

gtk: don't use gtk_window_set_titlebar if adwaita is enabled but it's older than 1.4.0 #4110

Merged
merged 1 commit into from
Dec 30, 2024

Conversation

jcollie
Copy link
Collaborator

@jcollie jcollie commented Dec 30, 2024

Fix #4097

Copy link
Collaborator

@tristan957 tristan957 left a comment

Choose a reason for hiding this comment

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

Thanks for fixing my mistake!

@tristan957
Copy link
Collaborator

It would be good to mention the discussion too.

@jcollie
Copy link
Collaborator Author

jcollie commented Dec 30, 2024

It would be good to mention the discussion too.

Yeah, once I get confirmation from the folks in #4097 that this actually works for them I'll update the description with relevant info.

@mitchellh
Copy link
Contributor

Failing CI is a flappy test. The rest looks good. Just advise when you want me to merge.

@jcollie jcollie force-pushed the fix-4097 branch 3 times, most recently from 4c0bc0b to 0c028e0 Compare December 30, 2024 20:23
@jcollie
Copy link
Collaborator Author

jcollie commented Dec 30, 2024

OK, I've gotten two reports that this fixes the issue so it's ready to merge.

@jcollie jcollie marked this pull request as ready for review December 30, 2024 20:59
@mitchellh mitchellh merged commit d548176 into ghostty-org:main Dec 30, 2024
21 checks passed
@github-actions github-actions bot added this to the 1.0.1 milestone Dec 30, 2024
@mitchellh mitchellh removed this from the 1.0.1 milestone Dec 30, 2024
mitchellh added a commit that referenced this pull request Dec 31, 2024
@jcollie jcollie deleted the fix-4097 branch January 3, 2025 01:12
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