Skip to content

Missing description of (what I consider unituitive) behavior of 'entry_marker' in the docs #3451

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.

Sign up for GitHub

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

Jump to bottom
Open
Weyaaron opened this issue Apr 18, 2025 · 0 comments · May be fixed by #3461
Open

Missing description of (what I consider unituitive) behavior of 'entry_marker' in the docs #3451

Weyaaron opened this issue Apr 18, 2025 · 0 comments · May be fixed by #3461
Labels
enhancement Enhancement to performance, inner workings or existent features

Comments

@Weyaaron
Copy link
Contributor

Is your feature request related to a problem? Please describe.
The problem I encountered is that I tried to follow the setup described in the developers_md and it did not work as intended. The picker was empty instead of showing my values. This has been solved by changing the table provided to 'entry_marker' to be indexed by integers. This worked.

Describe the solution you'd like
I would like the documentation to be altered such that this requirement is mentioned.
Describe alternatives you've considered
One could also alter the picker behavior such that this works on non-integer-indexed tables. I think it should work on these, but I am fine with adding documentation.

Additional context
I would be open to contributing a suggestion, I have had a small contribution to telescope in the past and know my way around the process.
@Weyaaron Weyaaron added the enhancement Enhancement to performance, inner workings or existent features label Apr 18, 2025
Weyaaron pushed a commit to Weyaaron/telescope.nvim that referenced this issue Apr 28, 2025
docs(developers.md): Add sentence to ensure integer indexing (nvim-te…
d32733f 
…lescope#3451)
@Weyaaron Weyaaron linked a pull request Apr 28, 2025 that will close this issue
Open
4 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement Enhancement to performance, inner workings or existent features
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

docs(developers.md): Add sentence to ensure integer indexing (#3451) Weyaaron/telescope.nvim
1 participant
@Weyaaron