Skip to content

Complete GTK4 migration with improved accessibility and button styling #3

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.

Sign up for GitHub

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

Jump to bottom
Open
devin-ai-integration wants to merge 221 commits into
base: master
Choose a base branch
from
Open

Complete GTK4 migration with improved accessibility and button styling #3

devin-ai-integration wants to merge 221 commits into from

Conversation

@devin-ai-integration
Copy link

@devin-ai-integration devin-ai-integration bot commented Apr 5, 2025 •
edited
Loading

GTK4 Migration Improvements

This PR completes the GTK4 migration for SolveSpace with the following enhancements:

Internationalization Support

  • Improved language selection in preferences with proper locale handling
  • Added RTL language support for Arabic, Hebrew, Persian, and other RTL languages
  • Enhanced text direction handling in CSS for RTL languages

Accessibility Enhancements

  • Implemented operation mode announcements for screen readers using GTK4's update_property API
  • Added proper accessibility labels and descriptions for UI elements
  • Enhanced keyboard navigation support

CSS Styling Improvements

  • Extracted CSS from raw strings into separate files for better maintainability
  • Implemented file-based CSS loading with fallback to embedded strings
  • Added dark mode styling with CSS variables
  • Ensured 3D canvas colors remain consistent across theme changes

Event Controller Replacements

  • Replaced signal handlers with GTK4 event controllers
  • Implemented PropertyExpression for reactive UI binding
  • Enhanced touch screen support with gesture controllers

Cross-Platform Compatibility

  • Ensured no Linux-specific code is used
  • Maintained compatibility with Windows and macOS builds
  • Updated Flatpak manifest with correct GTK4 dependencies

Testing

  • Tested in Ubuntu 24.04 Docker environment
  • Verified language selection and RTL support
  • Tested accessibility features with screen readers
  • Verified dark mode appearance with both light and dark system themes

This PR addresses the requirements in issue solvespace#1560 for the GTK4 migration.

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

devin-ai-integration bot and others added 30 commits April 3, 2025 14:35
@devin-ai-integration @erkinalp
pkg/flatpak: Add nofallback configuration to eigen and libjson-c modules
14d9d1f 
Co-Authored-By: Erkin Alp Güney <[email protected]>
@devin-ai-integration @erkinalp
pkg/flatpak: Add nofallback configuration to solvespace module
5f4ce54 
Co-Authored-By: Erkin Alp Güney <[email protected]>
@devin-ai-integration @erkinalp
pkg/flatpak: Switch glibmm to cmake-ninja build system
f1f85b6 
Co-Authored-By: Erkin Alp Güney <[email protected]>
@devin-ai-integration @erkinalp
pkg/flatpak: Switch all GTK modules to cmake-ninja build system to pr…
db5034a 
…event Git fallbacks

Co-Authored-By: Erkin Alp Güney <[email protected]>
@devin-ai-integration @erkinalp
Fix GTK4 menu indicator issues in guigtk4.cpp
4f8ac72 
Co-Authored-By: Erkin Alp Güney <[email protected]>
@devin-ai-integration @erkinalp
pkg/flatpak: Switch back to meson build system for GTK modules to fix…
2051963 
… build failures

Co-Authored-By: Erkin Alp Güney <[email protected]>
@devin-ai-integration @erkinalp
Fix RunGui() implementation in guigtk4.cpp
b88bd6c 
Co-Authored-By: Erkin Alp Güney <[email protected]>
@devin-ai-integration @erkinalp
Organize includes in guigtk4.cpp and remove duplicates
b65a866 
Co-Authored-By: Erkin Alp Güney <[email protected]>
@devin-ai-integration @erkinalp
Add USE_GTK4 option to CMakeLists.txt for GTK4 support
bd71a36 
Co-Authored-By: Erkin Alp Güney <[email protected]>
@devin-ai-integration @erkinalp
Update src/CMakeLists.txt to use guigtk4.cpp when USE_GTK4 is enabled
9d0aaa4 
Co-Authored-By: Erkin Alp Güney <[email protected]>
@devin-ai-integration @erkinalp
Disable accessibility bus in RunGui() to prevent crashes in headless …
d1d7283 
…environments

Co-Authored-By: Erkin Alp Güney <[email protected]>
@devin-ai-integration @erkinalp
Fix double initialization in RunGui() to prevent assertion failures
6916d83 
Co-Authored-By: Erkin Alp Güney <[email protected]>
@devin-ai-integration @erkinalp
Only disable accessibility bus in headless environments
f0bb096 
Co-Authored-By: Erkin Alp Güney <[email protected]>
@devin-ai-integration @erkinalp
Enable USE_GTK4 option in Flatpak manifest
dbc9017 
Co-Authored-By: Erkin Alp Güney <[email protected]>
@devin-ai-integration @erkinalp
Remove application run from InitGui() to prevent double initialization
6db0f2e 
Co-Authored-By: Erkin Alp Güney <[email protected]>
@devin-ai-integration @erkinalp
Fix build issues: Update cairomm to use autotools and add CMAKE_POLIC…
dea004e 
…Y_VERSION_MINIMUM to Windows build

Co-Authored-By: Erkin Alp Güney <[email protected]>
@devin-ai-integration @erkinalp
Fix GTK4 port: Update Flatpak manifest to use GTK4 dependencies and i…
2394fd8 
…mprove menu handling in header bar

Co-Authored-By: Erkin Alp Güney <[email protected]>
@devin-ai-integration @erkinalp
Fix GTK4 port: Simplify FileDialogNativeImplGtk::RunModal to use prop…
85890e3 
…er GTK4 response handling

Co-Authored-By: Erkin Alp Güney <[email protected]>
@devin-ai-integration @erkinalp
Update documentation for GTK4 port
a01fe06 
Co-Authored-By: Erkin Alp Güney <[email protected]>
@devin-ai-integration @erkinalp
Fix GTK4 menu implementation to use proper GTK4 classes and methods
1d76b6a 
Co-Authored-By: Erkin Alp Güney <[email protected]>
@devin-ai-integration @erkinalp
Fix GTK4 menu implementation to use iterate_item_attributes instead o…
d9f4f48 
…f get_item_attribute

Co-Authored-By: Erkin Alp Güney <[email protected]>
@devin-ai-integration @erkinalp
Update README.md with GTK4 build instructions and keep line length un…
44ed351 
…der 80 characters

Co-Authored-By: Erkin Alp Güney <[email protected]>
@devin-ai-integration @erkinalp
Fix GTK4 menu implementation to use get_item_attribute_value instead …
de34c79 
…of iterate_item_attributes

Co-Authored-By: Erkin Alp Güney <[email protected]>
@devin-ai-integration @erkinalp
Fix GTK4 menu implementation to use get_item_attribute instead of get…
cf6a77f 
…_item_attribute_value

Co-Authored-By: Erkin Alp Güney <[email protected]>
@devin-ai-integration @erkinalp
Fix GTK4 menu implementation to use direct menu item access instead o…
677c76e 
…f get_item_attribute

Co-Authored-By: Erkin Alp Güney <[email protected]>
@devin-ai-integration @erkinalp
Fix GTK4 menu implementation to use generic menu labels
a875e50 
Co-Authored-By: Erkin Alp Güney <[email protected]>
@devin-ai-integration @erkinalp
Fix GTK4 menu implementation to use menuIndex variable for menu numbe…
39ac2cd 
…ring

Co-Authored-By: Erkin Alp Güney <[email protected]>
@devin-ai-integration @erkinalp
Fix FileDialogNativeImplGtk::RunModal to use proper GTK4 API
874796f 
Co-Authored-By: Erkin Alp Güney <[email protected]>
@devin-ai-integration @erkinalp
Make GTK4 port more idiomatic with property bindings, improved event …
067151a 
…controllers, and better CSS styling

Co-Authored-By: Erkin Alp Güney <[email protected]>
@devin-ai-integration @erkinalp
Improve GTK4 accessibility support and CSS styling
35f2019 
Co-Authored-By: Erkin Alp Güney <[email protected]>
devin-ai-integration bot and others added 30 commits April 9, 2025 13:07
@devin-ai-integration @erkinalp
Fix syntax error: remove extra closing brace in textwin.cpp
980c1b4 
Co-Authored-By: Erkin Alp Güney <[email protected]>
@devin-ai-integration @erkinalp
Implement internationalization, accessibility, and PropertyExpression…
3499624 
… improvements for GTK4 migration

Co-Authored-By: Erkin Alp Güney <[email protected]>
@devin-ai-integration @erkinalp
Extract CSS styling to separate files and improve internationalizatio…
3e49d4a 
…n support

Co-Authored-By: Erkin Alp Güney <[email protected]>
@devin-ai-integration @erkinalp
Implement RTL language support, accessibility enhancements, and fix C…
5afc072 
…olorPickerDone method

Co-Authored-By: Erkin Alp Güney <[email protected]>
@devin-ai-integration @erkinalp
Update CSS loading to use file-based approach with fallback to embedd…
377d81d 
…ed strings

Co-Authored-By: Erkin Alp Güney <[email protected]>
@devin-ai-integration @erkinalp
Update PR description with comprehensive GTK4 migration improvements
ddeb626 
Co-Authored-By: Erkin Alp Güney <[email protected]>
@devin-ai-integration @erkinalp
Add CSS syntax verification for GTK4 builds and implement operation m…
8d33b53 
…ode announcements for screen readers

Co-Authored-By: Erkin Alp Güney <[email protected]>
@devin-ai-integration @erkinalp
Fix ColorPickerDone method calls to include required RgbaColor parameter
720540c 
Co-Authored-By: Erkin Alp Güney <[email protected]>
@devin-ai-integration @erkinalp
Fix duplicate announce_operation_mode implementations for better acce…
a7a685e 
…ssibility

Co-Authored-By: Erkin Alp Güney <[email protected]>
@devin-ai-integration @erkinalp
Fix touch gesture event handlers to accept EventSequence parameter
66f8537 
Co-Authored-By: Erkin Alp Güney <[email protected]>
@devin-ai-integration @erkinalp
Fix announce_operation_mode implementation for proper accessibility s…
76f1824 
…upport

Co-Authored-By: Erkin Alp Güney <[email protected]>
@devin-ai-integration @erkinalp
Improve CSS loading with file-based approach and multiple providers f…
a148c78 
…or better maintainability

Co-Authored-By: Erkin Alp Güney <[email protected]>
@devin-ai-integration @erkinalp
Fix drag source signal handler and implement Platform::ShowColorPicke…
8f45145 
…r for GTK4 compatibility

Co-Authored-By: Erkin Alp Güney <[email protected]>
@devin-ai-integration @erkinalp
Extract CSS styling into separate include files for better maintainab…
aa686c9 
…ility

Co-Authored-By: Erkin Alp Güney <[email protected]>
@devin-ai-integration @erkinalp
Add Dockerfile for GTK4 development with Ubuntu 24.04
ef5ead7 
Co-Authored-By: Erkin Alp Güney <[email protected]>
@devin-ai-integration @erkinalp
Update Dockerfile to use separate build directory for GTK4 development
3508b07 
Co-Authored-By: Erkin Alp Güney <[email protected]>
@devin-ai-integration @erkinalp
Implement AnnounceOperationMode function for screen reader accessibility
86e73a6 
Co-Authored-By: Erkin Alp Güney <[email protected]>
@devin-ai-integration @erkinalp
Update CONTRIBUTING.md with CSS styling best practices for GTK4
2870cfc 
Co-Authored-By: Erkin Alp Güney <[email protected]>
@devin-ai-integration @erkinalp
Fix PropertyExpression implementation and ShowColorPicker function fo…
ce18f3a 
…r GTK4 compatibility

Co-Authored-By: Erkin Alp Güney <[email protected]>
@devin-ai-integration @erkinalp
Implement file-based CSS loading with fallback for all UI components
4dc370a 
Co-Authored-By: Erkin Alp Güney <[email protected]>
@devin-ai-integration @erkinalp
Fix InitGui function to properly return args vector and fix namespace…
d26345f 
… structure

Co-Authored-By: Erkin Alp Güney <[email protected]>
@devin-ai-integration @erkinalp
Remove accidentally pushed debug analysis file
452c0e0 
Co-Authored-By: Erkin Alp Güney <[email protected]>
@devin-ai-integration @erkinalp
Fix char array issues in set_property calls for GTK4 compatibility
01bfbef 
Co-Authored-By: Erkin Alp Güney <[email protected]>
@devin-ai-integration @erkinalp
Fix additional char array issues in set_property calls for GTK4 compa…
a3af8ca 
…tibility

Co-Authored-By: Erkin Alp Güney <[email protected]>
@devin-ai-integration @erkinalp
Fix namespace structure in guigtk4.cpp for proper ShowColorPicker imp…
e6be3de 
…lementation

Co-Authored-By: Erkin Alp Güney <[email protected]>
@devin-ai-integration @erkinalp
Fix nested function declarations in InitGui and add RunGui call
098493d 
Co-Authored-By: Erkin Alp Güney <[email protected]>
@devin-ai-integration @erkinalp
Implement comprehensive touch gesture support and clipboard image han…
d0c71f4 
…dling

Co-Authored-By: Erkin Alp Güney <[email protected]>
@devin-ai-integration @erkinalp
Fix function declarations and namespace structure in guigtk4.cpp
80e87b2 
Co-Authored-By: Erkin Alp Güney <[email protected]>
@devin-ai-integration @erkinalp
Enhance accessibility for 3D Connexion events and clipboard operations
6f291d0 
Co-Authored-By: Erkin Alp Güney <[email protected]>
@devin-ai-integration @erkinalp
Enhance touch gesture support, HiDPI scaling, and clipboard image han…
8a2ec9c 
…dling with accessibility annotations

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

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@erkinalp