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

Flags to enable telescope preview and to choose picker #97

Merged
merged 3 commits into from Apr 13, 2021
Merged

Flags to enable telescope preview and to choose picker #97

merged 3 commits into from Apr 13, 2021

Conversation

AckslD
Copy link
Contributor

@AckslD AckslD commented Apr 13, 2021

closes #92

@soywod I also did some restructuring of the code around this and the lua file. Not necessary for the added feature but I found it a bit more readable. Feel free to let me know if you don't like this structure.

Copy link
Owner

@soywod soywod left a comment

Choose a reason for hiding this comment

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

I really like the way you refactored the input! I just have some minor changes related to coding style (see the review).

vim/autoload/himalaya/mbox.vim Outdated Show resolved Hide resolved
vim/lua/himalaya/mbox.lua Outdated Show resolved Hide resolved
vim/autoload/himalaya/mbox.vim Outdated Show resolved Hide resolved
@AckslD
Copy link
Contributor Author

AckslD commented Apr 13, 2021

Thanks for the feedback @soywod, I made some updates based on this

@soywod soywod merged commit 8a617c3 into soywod:master Apr 13, 2021
@soywod
Copy link
Owner

soywod commented Apr 13, 2021

Thank you so much ❤️

@AckslD AckslD deleted the 92-telescope-preview-enabled-flag branch April 13, 2021 15:17
@soywod soywod added enhancement New feature or request vim Related to the Vim plugin labels Apr 17, 2021
@soywod soywod added this to the v1.0.0 milestone Apr 17, 2021
@soywod
Copy link
Owner

soywod commented Apr 24, 2021

@AckslD I realized we forgot to update the vim readme about those new options!

@AckslD
Copy link
Contributor Author

AckslD commented May 1, 2021

Good point @soywod! Unfortunately I'm not able to contribute much at the moment :)

@soywod
Copy link
Owner

soywod commented May 8, 2021

I added a configuration section in the vim README with the options you introduced 😉

koalp pushed a commit to koalp/himalaya that referenced this pull request Sep 17, 2021
* Flags to enable telescope preview and to choose picker

* 4 spaces -> 2 space lua

* changes to align with repo code-style
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request vim Related to the Vim plugin
Projects
None yet
2 participants