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

Default builtin quickfix to use current list instead of last. #3393

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

nsoufian
Copy link

Description

This change updates Telescope quickfix command to default to opening the current quickfix list instead of the last quickfix list. The previous behavior caused confusion when navigating the quickfix stack (e.g., using :colder or :cnewer), as Telescope would always display the last quickfix list instead of the currently active one.

This update aligns Telescope's behavior with the user's expectation of viewing the active quickfix list. Users who wish to explicitly view the last quickfix list can still configure their query with appropriate parameters.

How Has This Been Tested?

Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list relevant details about your configuration.

  • Verified navigation between quickfix lists (:colder, :cnewer) works correctly with Telescope.
  • Tested default behavior to open the current quickfix list.
  • Tested explicit configuration to query the last quickfix list when needed.

Configuration:

  • Neovim version (nvim --version):
    NVIM v0.11.0-dev-1410+gc51bf5a6b2
    Build type: Release
    LuaJIT 2.1.1734355927
    Run "nvim -V1 -v" for more info
    
  • Operating system and version:
    5.15.167.4-microsoft-standard-WSL2 Ubuntu 22.04.1 LTS

Checklist:

  • My code follows the style guidelines of this project (stylua)
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation (lua annotations)

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