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

keybind with +[cmd] would not trigger view refreshing #1323

Closed
liangkarl opened this issue Mar 30, 2024 · 4 comments
Closed

keybind with +[cmd] would not trigger view refreshing #1323

liangkarl opened this issue Mar 30, 2024 · 4 comments

Comments

@liangkarl
Copy link

Hello,

I have a question about @ and +. Simply compare @ with +. bind stage co @git checkout %(file) would trigger view refreshing automatically, but bind stage co +git checkout %(file) would not.

I have no idea how to output the first line of the results and refresh the view at the same time. Does anyone could help to fix the problem? Thanks!

       @   Run the command in the background with no output.
       +   Run the command synchronously, and echo the first
           line of output to the status bar.
@liangkarl liangkarl changed the title keybind with + would not trigger refresh keybind with +[cmd] would not trigger view refreshing Mar 30, 2024
@koutcher
Copy link
Collaborator

koutcher commented Apr 1, 2024

When you use +, the view trigger is bypassed otherwise you wouldn't see the text in the status line.

@liangkarl
Copy link
Author

liangkarl commented Apr 1, 2024

Got it.
So, it's a dependent behavior with showing text and refreshing view, right?
Is it possible to adjust the sequence like refresh view before show the text in status bar?
Since I have tried to change to <, > and !, as well as adjust refresh-mode, I have no idea how to make it.

@koutcher
Copy link
Collaborator

koutcher commented Apr 2, 2024

You can't without code change. Try with #1324. It requires the removal of a report_clear() call in view.c that doesn't seem to do any harm and at least doesn't fail any test but more thorough non regression testing will be needed before considering merge.

koutcher added a commit to koutcher/tig that referenced this issue Apr 2, 2024
@liangkarl
Copy link
Author

Okay, thanks a lot!

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 a pull request may close this issue.

2 participants