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

Sorting icons not displaying properly #35

Closed
Shootertrex opened this issue Feb 26, 2020 · 6 comments · Fixed by #36
Closed

Sorting icons not displaying properly #35

Shootertrex opened this issue Feb 26, 2020 · 6 comments · Fixed by #36
Assignees
Labels
bug Something isn't working the way that is expected.

Comments

@Shootertrex
Copy link

Describe the bug

The arrows used to indicate that a column has been sorted does not fully appear.

To reproduce

  1. Start bottom
  2. Sort a process column e.g. cpu

Screenshots

Screenshot_20200225_212133

Screenshot_20200225_213157

Platform

Operating System:

OpenSUSE Tumbleweed

Terminal:

Konsole

Shell:

bash 5.0.11

Additional Details:

On another computer using Terminator (also bash), a similar issue happened, but the right half of the triangle didn't repaint until the terminal was resized (half pointed up, half pointed down and vice versa). Unfortunately I don't have screenshots of it.

@Shootertrex Shootertrex added the bug Something isn't working the way that is expected. label Feb 26, 2020
@ClementTsang
Copy link
Owner

@Shootertrex I'm able to replicate your Konsole issue and I have a fix... can't seem to replicate your Terminator issue though.

5d90c23 should fix the Konsole issue, however.

@ClementTsang
Copy link
Owner

Going to close since this should solve the Konsole issue and I can't replicate your terminator issue; feel free to reopen if this fix doesn't solve either though.

@Shootertrex
Copy link
Author

Thank you for the quick turnaround. I can confirm that it is fixed in Konsole, but it still occurs in Terminator for me. I was able to get screenshots today (occurs in both directions), but if you're unable to reproduce it yourself, they may not be useful.

1
2

I'm not sure if it's just something on my end then. It's not a big deal though.

@ClementTsang
Copy link
Owner

ClementTsang commented Feb 27, 2020

@Shootertrex if possible, can you send me any config files and font choice you're currently using on Terminator?

I'll open a new issue and look into trying to replicate this problem + fix it.

@ClementTsang
Copy link
Owner

ClementTsang commented Feb 27, 2020

@Shootertrex sorry to ask you again, but can you try cloning from master and see if #46 solves your problem (once it's merged)?

I was having similar issues with the arrow on macOS and this solved it, so it might also deal with your issue on Terminator.

@Shootertrex
Copy link
Author

@ClementTsang Building and running master didn't exactly solve the Terminator issue. I want to preface this with that this is occurring on my work machine inside of an older VM (Ubuntu 18.04), so there could possibly be some weirdness coming from that. With that said, it still happens but moving the mouse cursor within the horizontal line where the arrow is refreshes it now.

Interestingly, Terminator and Tilix (which this also occurs on for me) do not appear to be displaying a small, single character arrow, while Konsole and urxvt both display the character correctly. Changing the font did not help (Ubuntu Mono Regular -> Oxygen Mono Regular). Also interestingly, changing the font size actually fixed the issue in all fonts I tested (except Ubuntu Mono Regular) when set to 12 or below (13 was what I had the terminal at before). It's still the larger arrow, but it will fully repaint.

At this point, I'm not sure if the issue is coming from bottom or from the VM itself, as it displays correctly on my home machine (Konsole). I don't know if any of this extra information is helpful, but just in case.

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

Successfully merging a pull request may close this issue.

2 participants