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

Switch to using transient? #97

Open
galeo opened this issue Apr 4, 2020 · 2 comments
Open

Switch to using transient? #97

galeo opened this issue Apr 4, 2020 · 2 comments

Comments

@galeo
Copy link

galeo commented Apr 4, 2020

Thanks for this awesome package.

Instead of using hydra, maybe it is more elegant to implement the UI with transient.

@joostkremers
Copy link
Owner

Yes, it's something I've always wanted to look at. transient seems a much better fit because its use-case is exactly what pandoc-mode does: call a shell command with lots of different command line arguments.

The current hydra-based implementation has a big advantage (for me as the maintainer, I mean). If a new option or switch is added to pandoc, it usually takes only a single line of code to add support for it to pandoc-mode. It took a bit of work to set that up, but it works quite nicely, now. I will have to look at transient more closely to see if that can be made to work.

@galeo
Copy link
Author

galeo commented Apr 5, 2020

Hope to hear good news from you
soon. Thanks.

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

No branches or pull requests

2 participants