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

Add opn plugin #2005

Closed
wants to merge 1 commit into from
Closed

Add opn plugin #2005

wants to merge 1 commit into from

Conversation

MatthiasKunnen
Copy link
Contributor

I would like to add a plugin that integrates with a tool I made called opn.

example_open_pdf

It is a terminal application picker that works using the XDG desktop spec. It can be seen as an alternative to the nuke plugin provided by nnn that is out-of-the-box customized to what the user has installed.

One question I had while making the plugin is the name, do users often add the plugin dir to their path? If they do, this will cause an infinite loop where the opn script will call itself. I can change the name to opn_nnn or similar should this be desired.

If you have any questions, or even feature ideas, I'll happily answer them

@N-R-K
Copy link
Collaborator

N-R-K commented Feb 24, 2025

a tool I made called opn.

Looks cool. Though it seems like there isn't many users (yet). I'd recommend hosting the plugin inside opn repo under extra/nnn/ or some directory like that. If it gains a larger userbase then we can consider adding it here. ATM the plugin is too trivial and for a tool with not much userbase.

@MatthiasKunnen
Copy link
Contributor Author

That's fair, it is very new, I created it because I needed an application picker to use with nnn. I had hoped to share it so others could use it too

@N-R-K
Copy link
Collaborator

N-R-K commented Feb 25, 2025

I had hoped to share it so others could use it too

You can create a "discussion" under show and tell section for this purpose.

@N-R-K N-R-K closed this Feb 25, 2025
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.

2 participants