Skip to content

Fx selector: fix cut-off effect names in drop-down list for Linux#1596

Closed
ronso0 wants to merge 5 commits into
mixxxdj:2.1from
ronso0:fx-selector-fix
Closed

Fx selector: fix cut-off effect names in drop-down list for Linux#1596
ronso0 wants to merge 5 commits into
mixxxdj:2.1from
ronso0:fx-selector-fix

Conversation

@ronso0
Copy link
Copy Markdown
Member

@ronso0 ronso0 commented Apr 10, 2018

Deere/LateNight: shift effect names left, tick mark still visible
Shade: shift effect names left, tick mark hidden
Tango: revert bg-color hack, bring back tick mark

Tested with Ubuntu 14.04 & Ubuntu Studio 14.04 (xfce session).
This fix is only necessary for a scale factor of 100%.
Above 100% works well, too, since the tick mark is not scaled anyway and therefore all effect names have more horizontal space.
Below 100% it seems to work, too, although thte tick mark is not scaled.

rc1-fx-names-linux-deere_fixed

rc1-fx-names-linux-latenight_fixed

rc1-fx-names-linux-shade_fixed

rc1-fx-names-linux-tango_fixed

Please confirm this still looks good with KDE and other desktops

Comment thread res/skins/Shade/style-linux.qss Outdated
@@ -0,0 +1,10 @@
/* hack around text getting cut off with scaled checkbox on Macs with Retina screens */
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This comment is confusing in style-linux.qss

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

whoopsy..
anything else slipped through?

Comment thread res/skins/Tango/style.qss
background: #0081B7;
image: url(skin:/buttons/btn_fx_selector_tick.svg) no-repeat center center;
background: #0081B7; */
}
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'll keep all those comments. Helpful, also it took quite a while to gather those infos..

@Be-ing
Copy link
Copy Markdown
Contributor

Be-ing commented Apr 13, 2018

What is the problem you are trying to solve with this? Testing on my 1366 x 768 laptop with KDE, the current 2.1 branch does not cut off effect names. But with this branch, every skin but Tango has effect names cut off on the left:
image
image
image
image

@ronso0
Copy link
Copy Markdown
Member Author

ronso0 commented Apr 14, 2018

What is the problem you are trying to solve with this?

Oblivion? #1590 (comment)
It's looking ...not good on Ubuntu Studio (14.04 + 16.04) which is IMO the mainstream linux distro for media production, running xfce desktop. Regarding #1590 and other PRs where started for the sake of perfection...if effect selector is now working to everyone's content on the major OSs/distros, and if this PR makes it look worse on those, forget about it..
If not, WEffectSelector should be fixed so it works reliably across all OS.

Re Linux distros: interesting to see QAbstractItemView occupies the same width as the collapsed selector and expands downwards, while on xfce it expands beyond that and doesn't show scrollbars.

@Be-ing
Copy link
Copy Markdown
Contributor

Be-ing commented Apr 14, 2018

It's quite frustrating that the QComboBox styling works so differently on each OS and even with different desktop environments (different Qt themes?). I guess we're running into a bug in QComboBox, but there's no way that's getting fixed for Qt4. I think we're going to have to adjust these styles all over again for 2.2 when switching to Qt5 anyway, so stressing out any more for perfection right now doesn't seem worth it IMO. Unfortunately I don't think there's a (reliable) way to detect different distros or Qt themes or whatever it is that's causing the styling heterogeneity and apply a different style... so I guess leave it how it is? 🤷‍♂️

@ronso0
Copy link
Copy Markdown
Member Author

ronso0 commented Apr 14, 2018

so I guess leave it how it is?

I'm fine with that.

@Be-ing
Copy link
Copy Markdown
Contributor

Be-ing commented Apr 14, 2018

Okay, thanks for trying.

@Be-ing Be-ing closed this Apr 14, 2018
@ronso0 ronso0 deleted the fx-selector-fix branch May 21, 2018 15:39
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.

2 participants