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

Gtk4 breakage #4

Open
wants to merge 19 commits into
base: dev
Choose a base branch
from
Open

Gtk4 breakage #4

wants to merge 19 commits into from

Conversation

LaserEyess
Copy link
Owner

Tracking PR for all GTK4 related changes. This will be messy and it won't compile for a while.

@LaserEyess LaserEyess force-pushed the gtk4_breakage branch 3 times, most recently from 7cd867a to 0da91be Compare December 19, 2023 20:25
LaserEyess and others added 19 commits January 9, 2024 21:09
In the GObject clean up this was missed, there needs to a member of the
type of the parent class in a GObject subclass.
I didn't use this feature, I didn't really understand it either. What I
do know is that it gets in the way of GTK4, which is the current goal.
Unused for now
To be added back as a GMenuModel thing, but this was responsible for
handingly connection state in the toolbar/main window, i.e. what remotes
there were to connect to. Seems strange, that should likely be in the
client?

Also was responsible for the popup menus in trg-trackers-tree-view (??)
Much work to be done here, some TODOs are placed elsewhere
Unneeded in gtk4, especially with listviews coming
Deprecated in gtk4, but also a final class, so the gtk3 subclassing
doesn't work.

Breakage: no mimetype file icon (e.g. book, file, media) in torrent tree
dialogs. Not a big deal, this will be added back with columnviews
anyway.
This removes right click menus and messes with some formatting. State
selector needs to be completely re-imagined soon anyway.
This set up a right click menu for clicking on files. No menus, so we
have to get rid of this.
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