Skip to content

feat(marks): add navigation (next, previous, select) #1415

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

Merged
merged 1 commit into from
Jul 16, 2022

Conversation

kyazdani42
Copy link
Member

@kyazdani42 kyazdani42 commented Jul 11, 2022

This feature will edit the file if the node is a file, and focus the folder in the tree if the node is a folder.

Added:

  • next mark
  • previous mark
  • select mark (vim.ui.select)

implements part of #1413

@kyazdani42 kyazdani42 requested a review from alex-courtis July 11, 2022 12:32
@kyazdani42 kyazdani42 force-pushed the feat/add-mark-navigation branch 2 times, most recently from 3041d15 to 8ce1931 Compare July 11, 2022 12:34
@kyazdani42 kyazdani42 mentioned this pull request Jul 11, 2022
8 tasks
@kyazdani42 kyazdani42 force-pushed the feat/add-mark-navigation branch from 8ce1931 to 3efc241 Compare July 11, 2022 12:44
@kyazdani42 kyazdani42 changed the title feat(marks): add navigation next and previous feat(marks): add navigation (next, previous, select) Jul 11, 2022
@kyazdani42 kyazdani42 force-pushed the feat/add-mark-navigation branch 6 times, most recently from dbeb11c to 6f24b60 Compare July 11, 2022 15:06
@kyazdani42 kyazdani42 force-pushed the feat/add-mark-navigation branch from 6f24b60 to f446831 Compare July 12, 2022 07:36
Copy link
Member

@alex-courtis alex-courtis left a comment

Choose a reason for hiding this comment

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

Tested OK:

  • file and folder
  • from nvim-tree
  • from buffer
  • update_focused_file.enabled
  • 0, 1, n marks

Type number and <Enter> or click with the mouse (q or empty cancels): is odd. I cannot make mouse selection work in a clean setup or my full setup. Not really related.

An option might be to use the location list to do the navigation, however that would probably result in a more (complicated) implementation with little benefit.

@kyazdani42 kyazdani42 merged commit 89becc7 into master Jul 16, 2022
@kyazdani42 kyazdani42 deleted the feat/add-mark-navigation branch July 16, 2022 08:40
Almo7aya pushed a commit to Almo7aya/nvim-tree.lua that referenced this pull request Oct 11, 2022
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