Skip to content

Only refresh flags when switching to normal buffers#6

Open
BenMotz wants to merge 1 commit into
PhilRunninger:masterfrom
BenMotz:fix-focus-with-telescope
Open

Only refresh flags when switching to normal buffers#6
BenMotz wants to merge 1 commit into
PhilRunninger:masterfrom
BenMotz:fix-focus-with-telescope

Conversation

@BenMotz

@BenMotz BenMotz commented Jul 23, 2026

Copy link
Copy Markdown

This resolves a bug when using Telescope with NerdTree and this plugin, where input focus ends up in the wrong part of the Telescope window.

What happens at present

  1. Telescope is opened, creating several windows
  2. The first of these is created, causing BufEnter to fire, running the autocmd in this plugin
  3. The plugin records the window number of the buffer
  4. Switches to the NerdTree window (to refresh the flags)
  5. Telescope creates some more windows, which invalidates the window number that the plugin recorded
  6. When the plugin restores focus, it restores it to the wrong window, and the input focus in Telescope ends up in the wrong place.

Fix by only calling RefreshFlags() for normal editable buffers, where buftype = ''

This resolves a bug when using Telescope with NerdTree and this plugin,
where input focus ends up in the wrong part of the Telescope window.

What happens at present
1. Telescope is opened, creating several windows
2. The first of these is created, causing BufEnter to fire, running the
   autocmd in this plugin
3. The plugin records the window number of the buffer
4. Switches to the NerdTree window (to refresh the flags)
5. Telescope creates some more windows, which invalidates the window
   number that the plugin recorded
6. When the plugin restores focus, it restores it to the wrong window,
   and the input focus in Telescope ends up in the wrong place.

Fix by only calling RefreshFlags() for normal editable buffers, where
buftype = ''
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.

1 participant