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

Vimode: implement fold in vimode plugin #1350

Merged
merged 1 commit into from
May 25, 2024

Conversation

scresto09
Copy link
Contributor

Supported commands are:
za / zo / zc toggle / open / close fold on one level of folding
zA / zO / zC toggle / open / close fold on all folding levels
zR / zM open / close all folds

@scresto09
Copy link
Contributor Author

Previous pull request #1327

Copy link
Member

@techee techee left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good apart from the several documentation-related changes below.

Once fixed, you can squash the result to a single commit. (And be careful about what git does ;-)

vimode/README Outdated
zR zR set 'foldlevel' to the deepest fold
za za open a closed fold, close an open fold
zc zc close a fold
zo zo open fold
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could you move this block to the same place as in index.txt, i.e. under section "2.5 commands starting with z"? And also within this section to the same place as in index.txt? (of course taking into account that some commands are missing)

vimode/README Outdated
@@ -96,7 +96,7 @@ This is an incomplete list of known limitations of the plugin:
* named registers and related commands are not implemented
* Ctrl+X mode is not implemented
* marks are not implemented
* fold commands are not implemented
* most fold commands are not implemented
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'd say "many" would be better as the most important ones are implemented now.

#include "cmds/fold.h"
#include "utils.h"

/* fold command does not depend on state of parents */
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'd say the comments above this define and the 2 other below is useless now and can be removed.

@scresto09
Copy link
Contributor Author

OK I did it

@techee techee merged commit 3e38966 into geany:master May 25, 2024
2 checks passed
@techee
Copy link
Member

techee commented May 25, 2024

Looks good, merged. Thanks a lot for all your contributions!!!

@scresto09
Copy link
Contributor Author

Thanks

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants