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

Display glitch when switching the cursor to block #95

Closed
ehjmx opened this issue Jun 14, 2023 · 6 comments
Closed

Display glitch when switching the cursor to block #95

ehjmx opened this issue Jun 14, 2023 · 6 comments
Labels
bug Something isn't working
Milestone

Comments

@ehjmx
Copy link

ehjmx commented Jun 14, 2023

At the request of @raphamorim I create this new issue.

I'm on macOS 13.4 and I'm using the canary build

canary-4935590abfe47dab24d53dc1e2268a790b4b94fb

There is display glitch when I switch to block for the cursor.

Cursor underscore looks like this
cursor_as_underscore

Cursor as block the block uses two lines
![cursor_as_block](https://github.com/raphamorim/rio/assets/75489369/7928e24b-1717-4879-8e8b-3452c8a91fa0

Version 0.0.61 worked fine. Ffter updating to the canary version and making the necessary modifications to the config file even 0.0.61 did show this glitch after reverting back and using the old config.toml.

Originally posted by @ehjmx in #92 (comment)

@raphamorim raphamorim added this to the 0.0.7 milestone Jun 14, 2023
@raphamorim raphamorim added the bug Something isn't working label Jun 14, 2023
raphamorim added a commit that referenced this issue Jun 15, 2023
@ehjmx
Copy link
Author

ehjmx commented Jun 16, 2023

With the canary-a891a800901db9431e68c32c8f03a300aea06e0e build the cursor glitches are gone and so is the sometimes inconsistent use of the terminal height and width.

For example ls used the full height and width but editors like joe, cmd line mail client aerc and artuin shell history only used the initial space that the terminal had when it was launched.

I was about to create an issue on this once work is done for today.

raphamorim added a commit that referenced this issue Jun 16, 2023
@raphamorim
Copy link
Owner

raphamorim commented Jun 16, 2023

That's great @ehjmx that fixed in the a891a80.

I am also writing tests for the text styles in the renderer, so will avoid breaking this occasionally.

Regarding the terminal height and width is that still happening? Would you mind exemplify a bit more?

@ehjmx
Copy link
Author

ehjmx commented Jun 18, 2023

The issue with the terminal heigh and width is still occassionally occuring, especially when using the 34 inch Monitor with his native 5k resolution.

This is the size of Rio when I open it
Bildschirmfoto 2023-06-18 um 10 46 14

Now I change the size
Bildschirmfoto 2023-06-18 um 10 47 08

When I now open e.g. joe editor, atuin or aerc only the above initial size is used in the bigger window.
When I use the vim, that comes with macOS or micro editor they use the full bigger window but at the cost of not using the theme that I set for Rio. Could this be related to issue #98 and/or because I set
env-vars = ['SHELL=/opt/local/bin/bash']
in my config file to use bash with Rio while I still have zsh as my default shell?

Using canary canary-2316d6de8782f6eaa94121c508c7798b303ca415 does help on the build in retina display on the MacBookPro M2 2022, macOS 13.4. Tomorrow, Monday, I can check if the whole heigth and width are also used on my 5k Monitor.

@raphamorim
Copy link
Owner

Thank you @ehjmx for the detailed issue, this problem was happening before right? (Asking because I have been doing a major refactor of the renderer to port to WASM)

@ehjmx
Copy link
Author

ehjmx commented Jun 18, 2023

Yes @raphamorim the issue was before the canary build I mentioned. The refactoring did fix it. However I can only confirm this for the build in Retina display. For the 5k monitor at work I will check tomorrow.

@raphamorim
Copy link
Owner

Tested in a 5k monitor using v0.0.8 and it does look ok, closing this issue but let me know anything.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants