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.

Sign up for GitHub

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

Jump to bottom

Eliding is needed sometimes #99

Closed
tsujan opened this issue Oct 26, 2018 · 0 comments · Fixed by #163
Closed

Eliding is needed sometimes #99

tsujan opened this issue Oct 26, 2018 · 0 comments · Fixed by #163
Projects
Issues Issues

Comments

@tsujan
Copy link
Member

tsujan commented Oct 26, 2018

Eliding is better than showing a horizontal scrollbar suddenly, when the text is too long.

Expected Behavior

Elide the text when needed.

Current Behavior

A scrollbar is suddenly shown, which seems neither practical nor professional:

eliding

System Information

Latest git version(s)
@agaida agaida added this to Needs triage in Issues Oct 26, 2018
@agaida agaida added this to To do in PRs Nov 5, 2018
@agaida agaida added this to To do in Issues Nov 25, 2018
@agaida agaida removed this from To do in PRs Nov 30, 2018
@agaida agaida moved this from Needs triage to Medium priority in Issues Feb 22, 2019
tsujan added a commit that referenced this issue Aug 16, 2020
@tsujan
GUI fixes: Elide playback name label if needed and…
3b246f5 
If the name is too long, elide it centrally, so that a horizontal scrollbar isn't needed (and isn't shown).

Also, don't force a height on progressbars because some widget styles may not show any bar with a forced height.

Closes #99 and closes #75
tsujan added a commit that referenced this issue Aug 16, 2020
@tsujan
GUI fixes: Elide playback name label if needed and…
0ae1420 
If the name is too long, elide it centrally, so that a horizontal scrollbar isn't needed (and isn't shown).

Also, don't force a height on progressbars because some widget styles may not show any bar with a forced height.

Closes #99 and closes #75
@tsujan tsujan mentioned this issue Aug 16, 2020
Merged
Issues automation moved this from Medium priority to Closed Aug 19, 2020
Issues automation moved this from To do to Done Aug 19, 2020
@tsujan tsujan mentioned this issue Oct 7, 2020
Closed
sandsmark pushed a commit to sandsmark/pavucontrol-qt that referenced this issue Feb 12, 2021
@tsujan @sandsmark
Fixed version of upstream patch
36e4714 
    GUI fixes: Elide playback name label if needed and…

    If the name is too long, elide it centrally, so that a horizontal scrollbar isn't needed (and isn't shown).

    Also, don't force a height on progressbars because some widget styles may not show any bar with a forced height.

    Closes lxqt#99 and closes lxqt#75
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
Issues
  
Closed
Issues
  
Done
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

GUI fixes: Elide playback name label if needed and… lxqt/pavucontrol-qt
1 participant
@tsujan