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

Convert the keyring unlock dialog to a clutter dialog #12452

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

JosephMcc
Copy link
Contributor

No description provided.

@fredcw
Copy link
Contributor

fredcw commented Oct 17, 2024

@JosephMcc Please excuse my ignorance but I'm curious about what the rationale is for moving from gtk dialogs to clutter dialogs. I thought that Clutter was in maintenance mode and has less compatibility with wayland, which it why GNOME it transitioning to gtk4 for it desktop UI? ChatGPT "informs" me:

While Wayland support could be improved in Cinnamon over time, it would likely require moving away from Clutter to something like GTK 4 or another modern rendering library that supports Wayland natively.

Also, is this an indication that @clefebvre planned to rewrite the cinnamon panel in gtk?

@JosephMcc
Copy link
Contributor Author

I thought that Clutter was in maintenance mode and has less compatibility with wayland, which it why GNOME it transitioning to gtk4 for it desktop UI?

The standalone version of Clutter is pretty much dead. The versions that are internal to mutter and muffin certainly are not. GNOME is not moving it's desktop UI to Gtk4. It relies very heavily on clutter for drawing all elements. Much more than Cinnamon does currently. This actually gives us a little more control over some UI elements ourselves instead of relying on dialogs that might be supplied by something else. And doing all of them keeps things consistent.

ChatGPT "informs" me:

LOL

Also, is this an indication that @clefebvre planned to rewrite the cinnamon panel in gtk?

That's more a brainstorm than anything else. I've even experimented with it in the past. But that would put even more reliance on Gtk.

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.

2 participants