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

RFC: Maximize horizontal use of terminal view #80

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

MoSal
Copy link
Contributor

@MoSal MoSal commented Jan 25, 2024

A couple of proposed changes on top of #79. This is scratching an itch I always had with cosmic-term. But others may consider the changes proposed here overkill.

Scrollback Changes

  • Don't clamp cols to before the scrollbar rectangle.
  • Auto-hide the scrollbar if a pane is not focused, or if there is no mouse activity.

Text changes

Slightly increase font size to get rid of the spare pixels that don't fit into a column at the end of a line.

Screenshots

These are taken with OneHalfDark on Light for clarity.

Current

master

With #79

no_wc

With scrollback changes

no_scroll_reserve

With font size increase

This fixes an issue that also exists in alacritty. But I never saw it in the previous terminals I used.

max_hor

 * Presumably, the TODO about window resizing interfering with scrolling
   is not applicable anymore with the existence of padding from the pane
   grid container. So, set `core.window.content_container` to false.

 * Pane grid container didn't follow the selected theme. So, set style
   on it to fix that.

Signed-off-by: Mohammad AlSaleh <CE.Mohammad.AlSaleh@gmail.com>
 This gets rid of the spare pixels at the end of lines that don't fit
 into a column.

Signed-off-by: Mohammad AlSaleh <CE.Mohammad.AlSaleh@gmail.com>
 * Don't clamp cols to before the scrollbar rectangle.
 * Auto-hide the scrollbar if a pane is not focused, or if there is
   no mouse activity.

Signed-off-by: Mohammad AlSaleh <CE.Mohammad.AlSaleh@gmail.com>
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.

None yet

1 participant