Skip to content

Conversation

@devin-ai-integration
Copy link

This PR implements the GTK4 migration as outlined in issue solvespace#853.

Changes

  • Created new guigtk4.cpp file based on guigtk.cpp
  • Updated CMakeLists.txt to conditionally include guigtk4.cpp when GTK4 is available
  • Updated menu implementation to use Gio::Menu and Gio::SimpleAction
  • Updated event handling to use event controllers (GestureClick, EventControllerMotion, etc.)
  • Updated window management code for GTK4 compatibility
  • Updated file dialogs to use GTK4 APIs
  • Updated application initialization to use Gtk::Application instead of Gtk::Main

Notes

  • The code has been built successfully with GTK3, but couldn't be tested with GTK4 libraries as they weren't available in the standard Ubuntu repository
  • The changes follow the migration checklist from issue Prep for GTK4 solvespace/solvespace#853
  • This implementation should be tested by someone with GTK4 libraries available

Link to Devin run: https://app.devin.ai/sessions/82a2b848bf614a0caaa05d68083e1438
Requested by: Erkin Alp Güney

@devin-ai-integration
Copy link
Author

🤖 Devin AI Engineer

I'll be helping with this pull request! Here's what you should know:

✅ I will automatically:

  • Address comments on this PR. Add "(aside)" to your comment to have me ignore it.
  • Look at CI failures and help fix them

Note: I can only respond to comments from users who have write access to this repository.

⚙️ Control Options:

  • Disable automatic comment and CI monitoring

devin-ai-integration bot and others added 15 commits April 1, 2025 19:09
…hawChoices, and CheckForUntitledFile methods

Co-Authored-By: Erkin Alp Güney <[email protected]>
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.

1 participant