Git commit #169
Draft
Git commit #169
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
We have a list of modified files, and basic layout. Next step is displaying the diff of current file - then link the preview with content of file.
When selected - a file, the diff of that file is displayed. We use the Qutepart, same as the main editor. That editor also shows the syntax of diff. I should use the text editor plugin to create the editor. This way the settings would follow, and double click would work (open the correct file).
I can revert a single item, and multiple is coded, but not tested yet.
1) When running git - return also the exit code. 2) (fix code compilation) 3) If on revert this fails - do not update file list. 4) Handle revert of single, and multiple file.
If you open a commit window, it will get restored when the app restarts.
There are some regressions on finding the repo root.
When the commit form now queries the text editor plugin, and creates the mdi editor from it. Now the settings of the editors, will be used for the diff viewer. Problems: - Clicking a diff, does not open the corresponding file.
I just needed to setup the mdi host on the editor.
The revert comand asks for confirmation.
I tested a simple commit on a dummy project, all seems to work.
Original code used ranges, which is not properly supported on Ubuntu 24.04. Use older code, which IMHO event looks nicer.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This PR adds ability to commit within the IDE. I have been using it locally - so the basics are on. I am working on usability and cleanups.