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

[Feature] Secondary key-binding U removing newlines #43

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

Conversation

tkapias
Copy link

@tkapias tkapias commented Feb 29, 2024

Intro

I'm using Neomutt TUI email client.
The main issue was, for me, that the internal pager does not rely on the terminal to wrap the long Urls.
As explained in this issue, it does insert newlines in lines to wrap them, which break most url extraction tools like the current tmux-fzf-url or the perl script in the URxvt terminal.

Feature

This PR introduce a new feature to work around this issue by removing every newlines in the captured content from Tmux, before the regex parsing.

  • This new feature is not an option but a secondary key-binding, by default 'U'.

I separated it in another key-binding because the captured urls might wrongly expand in some cases, e.g, if an Url matches perfectly the screen width and is followed by another string in the beginning of the next line.

But in cases like Neomutt it's a perfect solution.

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