fix: adw 1.6 deprecations #479
Conversation
|
Thanks for PR! Since this is part of the application itself, I am kinda surprised you are contributing to something that Tuba doesn't use 😉 I haven't looked closely at code yet, but one thing I think you might have missed is bumping minimal required libadwaita version in meson. |
I 💖 Clapper!
Done! I also bumped gtk's to libadwaita 1.5's minimum |
AdwMessageDialog is deprecated on libadwaita 1.6. Additionally, replace GtkEntry with AdwEntryRow to better follow the HIG.
AdwAboutWindow is deprecated on libadwaita 1.6.
GeopJr
force-pushed
the
fix/adw-1-6-deprecations
branch
from
448e84e
to
5c0b73f
Compare
|
Hi. Sorry for late reply (I was on vacation break last month). This PR looks good to me. Thanks! Although, if its ok with you, I will hold it off for now until I have time to work on AdwWindow ports (as I am working on some other Clapper feature right now), since currently this causes criticals (https://gitlab.gnome.org/GNOME/libadwaita/-/issues/912). There should be both this + AdwWindows in a single release of Clapper and I haven't decided yet if I will make a release before or after working on AdwWindow changes. |
Libadwaita 1.6 is around the corner so I migrated some dialogs to the non-deprecated versions:
AdwAboutWindow => AdwAboutDialog
AdwMessageDialog => AdwAlertDialog
I also changed the uri-dialog's GtkEntry to AdwEntryRow to better match the HIG
My C is a bit rusty, let me know if I should change something or if I should split this PR into smaller ones!
Additionally, there's another change that should be done but it's a bit too much for me to do, the main window should become AdwWindow / AdwApplicationWindow to take full advantage of the dialogs. Those windows can display the dialogs inside of them instead of creating new ones and making them modals, which has the added bonus of the dialogs becoming bottom-sheets on narrow window sizes to better fit mobile devices