Skip to content

Conversation

@shanesmith
Copy link
Contributor

Could easily be a plugin but it felt like it might belong here.

The action is style as focus:x for any future actions that might also set the focus.

A default mapping of p is added.

Copy link
Owner

@lambdalisue lambdalisue left a comment

Choose a reason for hiding this comment

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

Thanks for the contribution! Please remove the default mapping and add documentation for that mapping in doc/fern.txt 👍

nmap <buffer><nowait> l <Plug>(fern-action-expand)
nmap <buffer><nowait> h <Plug>(fern-action-collapse)
nmap <buffer><nowait> i <Plug>(fern-action-reveal)
nmap <buffer><nowait> p <Plug>(fern-action-focus:parent)
Copy link
Owner

Choose a reason for hiding this comment

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

Please remove this. Let users to define the mapping for that.

@shanesmith shanesmith force-pushed the action-focus-parent branch from 414cda8 to 508dd19 Compare March 24, 2021 13:51
@shanesmith
Copy link
Contributor Author

The docs of course, my bad! I've also removed the mapping.

@shanesmith shanesmith force-pushed the action-focus-parent branch from 508dd19 to bf675a0 Compare March 24, 2021 14:05
Copy link
Owner

@lambdalisue lambdalisue left a comment

Choose a reason for hiding this comment

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

LGTM

@lambdalisue lambdalisue merged commit 3f9f195 into lambdalisue:master Mar 25, 2021
@lambdalisue
Copy link
Owner

Merged and released as v1.34.0 👍 Thanks for the contribution 🎉

@shanesmith
Copy link
Contributor Author

Much appreciated!

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