-
-
Notifications
You must be signed in to change notification settings - Fork 80
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 skeleton #2549
base: main
Are you sure you want to change the base?
Gtk4 skeleton #2549
Conversation
# Conflicts fixed : # libcore/DndHandler.vala # meson.build # src/View/AbstractDirectoryView.vala # src/View/Sidebar/BookmarkRow.vala
|
Because I want this PR not to require detailed review as a whole because the diff will be large. If the merged PRs have already been independently reviewed at least for coding rationality then this PR will only need reviewing for overall functionality and appearance. |
# Conflicts fixed: # filechooser-portal/Main.vala # libcore/AbstractSlot.vala # libcore/FileUtils.vala # plugins/pantheon-files-trash/plugin.vala # src/Dialogs/ChooseAppDialog.vala # src/View/DirectoryNotFound.vala # src/View/Window.vala
The intention of this is to provide an intermediate branch which compiles (but does not necessarily run atm) with Gtk4 to which small branches may be pushed porting different aspects of the code. This allows for a series of manageable reviews and approvals.
The non-compiling code has the for the most part been commented out except where a very simple fix is possible or it was not possible to fix by just commenting code out.
Branches pushed to this branch should incrementally replace commented out code with working Gtk4 code with as small a diff as practical.
As of 745e1a7, this compiles and runs with no terminal messages - but only shows an empty window.
@danirabbit Is there a way of having PRs that are to be merged into this branch require at least 1 approving review?
--PRs be merged--