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 request: Visible bell (vbell) after command line #456

Open
bb010g opened this issue Jun 5, 2024 · 4 comments
Open

Feature request: Visible bell (vbell) after command line #456

bb010g opened this issue Jun 5, 2024 · 4 comments
Labels
NYI/NewFeat Not yet implemented or New Feature

Comments

@bb010g
Copy link

bb010g commented Jun 5, 2024

Currently, the visible bell can overwrite potentially important parts of my terminal's scrollback. However, I would like to not completely disable the visible bell. Could an option be added to allow the visible bell to render under the current command line, similar to how completions are rendered, such that terminal scrollback can be preserved?

@akinomyoga akinomyoga added the NYI/NewFeat Not yet implemented or New Feature label Jun 9, 2024
@akinomyoga
Copy link
Owner

Sorry for the delay. I thought I could implement it soon, but I was working on other parts. I'll add the feature.

BTW, the reason that the messages of the visible bell is shown outside the drawing area of the line editor was that I initially wanted to make the visible bell available also from subshell processes. Since the drawing area of the line editor is under the control of the main shell, if a subshell changes its content, the layout and the rendering of the drawing area would be broken. However, the visible bell turned out to be only used by the main shell after ble.sh is matured, so we can safely render the visible bell within the drawing area of the line editor.

@akinomyoga
Copy link
Owner

I implemented it in commit fe85e0d. The location of the visible bell can be configured using bleopt vbell_align.

  • This time, I added a new option panel to the possible values of vbell_align, though it's not turned on by default.
  • I also changed the default horizontal alignment from left to right (which I thought I did several years ago, but I realized I actually didn't). Now, the messages are shown on the top right corner of the terminal by default.

You can show the visible-bell messages below the command line by using the following setting:

# blerc

bleopt vbell_align=panel

Since this is a new feature, there are still some parts unstable. @bb010g Could you update ble.sh by running ble-update in a ble.sh session and check the behavior under the setting bleopt vbell_align=panel?

@akinomyoga
Copy link
Owner

@bb010g Could you check the behavior? If not, can I close the issue?

@akinomyoga
Copy link
Owner

@bb010g ?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
NYI/NewFeat Not yet implemented or New Feature
Projects
None yet
Development

No branches or pull requests

2 participants