[neon] [Bug 362767] High DPI is messed up after Qt 5.6 update

[Greg Varsanyi via KDE Bugzilla]

https://bugs.kde.org/show_bug.cgi?id=362767

Greg Varsanyi  changed:

   What|Removed |Added

 Resolution|WAITINGFORINFO  |FIXED
 Status|NEEDSINFO   |RESOLVED

--- Comment #11 from Greg Varsanyi  ---
A lot has changed since I opened this ticket. At this point (Qt 5.7) defaults
work well mostly, no need for extra trickery with env variables and such.

closing as fixed.

-- 
You are receiving this mail because:
You are watching all bug changes.

[neon] [Bug 362767] High DPI is messed up after Qt 5.6 update

[Greg Varsanyi via KDE Bugzilla]

https://bugs.kde.org/show_bug.cgi?id=362767

--- Comment #10 from Greg Varsanyi  ---
(In reply to Weng Xuetian from comment #9)
> IIRC, though QT_DEVICE_PIXEL_RATIO is still there, it's not doing the exact
> same thing as it is in 5.5 . For now I'm using a combination of
> QT_AUTO_SCREEN_SCALE_FACTOR and QT_SCREEN_SCALE_FACTORS to achieve the same
> effect.

what combination do you use? and how do you make it system wide exactly?

> And kwin/plasma shouldn't be affected since they are not using it anyway, if
> you found kwin/plasma is scaled, probably your distro need to compile
> kwin/plasma against Qt 5.6 to make some qt5.6 compile time specific check
> work.

plasma and kwin seem to use the font size, around the idea of `1 line height in
rendered px = perceivedDPI / 96 * referenceFontBoundaryHeight` where
perceivedDPI seems to come from QT_FONT_DPI / fonts.conf DPI value / X.org
global DPI

--

how do I check what Qt version are plasmashell or kwin_x11 compiled against?
for apps it's shown on the about/version panel, but `plasmashell -v` only gives
me plasma version.

-- 
You are receiving this mail because:
You are watching all bug changes.

[neon] [Bug 362767] High DPI is messed up after Qt 5.6 update

[Greg Varsanyi via KDE Bugzilla]

https://bugs.kde.org/show_bug.cgi?id=362767

--- Comment #8 from Greg Varsanyi  ---
(In reply to David Edmundson from comment #5)
> For Case #1:
>  - qt now scales up font DPI for some reason, you can counter that by
> explicitly also setting
> QT_FONT_DPI to be half/double (not sure which way round it is) 
> 
> For Case #3
> 
> Since Plasma 5.5 we do have code to disable internal high DPI managing on
> Qt5.6
> 
> However, it requires kwin/plasma also be *compiled* with Qt5.6
> 
> #if (QT_VERSION >= QT_VERSION_CHECK(5, 6, 0))
> QCoreApplication::setAttribute(Qt::AA_DisableHighDpiScaling);
> #endif

playing with QT_FONT_DPI gets me the closest.
a combination of QT_FONT_DPI=96 and QT_DEVICE_PIXEL_RATIO=2 (and not
setting/unsetting any new Qt 5.6 env vars) gives me consistent look and feel
across the *apps*, but not plasmashell and kwin - they go half sized.
it would seem that plasmashell and kwin read the font dpi value and apply that
literally as pixels.
that can also be mitigated by starting these two (and only these two) with
QT_FONT_DPI=192.

So, using a deprecated and a falsified env var - with some exception handling -
mostly gets us there.

--

i wonder why did you guys decided to disable the native scaling.

-- 
You are receiving this mail because:
You are watching all bug changes.

[neon] [Bug 362767] High DPI is messed up after Qt 5.6 update

[Greg Varsanyi via KDE Bugzilla]

https://bugs.kde.org/show_bug.cgi?id=362767

--- Comment #2 from Greg Varsanyi  ---
(In reply to Harald Sitter from comment #1)
> Which edition of neon are you on? I can poke some rebuilds so we'll know for
> sure.

dev/stable

btw, new packages seem to be compiled against qt 5.6 but all issues remain the
same.

Unless there is a tricky combination of ENV vars I haven't figured out, I'm
afraid this is a development issue where deprecated logic/code has to be
updated all across the code base to play better with Qt 5.6 high dpi logic.

-- 
You are receiving this mail because:
You are watching all bug changes.

[neon] [Bug 362767] New: High DPI is messed up after Qt 5.6 update

[Greg Varsanyi via KDE Bugzilla]

https://bugs.kde.org/show_bug.cgi?id=362767

Bug ID: 362767
   Summary: High DPI is messed up after Qt 5.6 update
   Product: neon
   Version: unspecified
  Platform: Other
OS: Linux
Status: UNCONFIRMED
  Severity: normal
  Priority: NOR
 Component: general
  Assignee: n...@kde.org
  Reporter: gvarsa...@gmail.com

Screen scaling (kcm_kscreen) still sets the deprecated QT_DEVICE_PIXEL_RATIO,
the new QT_AUTO_SCREEN_SCALE_FACTOR confuses kwin_x11 and plasmashell really
bad.

I wonder if this has to do with packages still being compiled against Qt 5.5.1

Reproducible: Always

Steps to Reproduce:
1. Grab a high DPI screen
2. Set screen scaling (Display Configuration > Screen Scaling > x2)
3. Log out and back in

or

1. Update to latest with Qt 5.6 on a system with high DPI configured.

Actual Results:  
case #1 (using QT_DEVICE_PIXEL_RATIO=2)
double (4x instead of 2x scaled) plasma parts and apps

case #2 (unset QT_DEVICE_PIXEL_RATIO)
some items are not sized up (fonts define minimum size), icons are too small
(1x instead of 2x)

case #3 (using QT_AUTO_SCREEN_SCALE_FACTOR=1)
kwin_x11 provides small (1x) decorations, has placement issues and changes the
height of all windows to the minimum possible

Expected Results:  
Scale to 2x consistantly
In an ideal word: make use of Qt 5.6 automatic screen scale factor

I realize this is partly of a Plasma development issue, but setting the
environment variables and compiling against Qt 5.6 might be a packaging thing.

-- 
You are receiving this mail because:
You are watching all bug changes.

[plasmashell] [Bug 360333] New system tray: icons do not scale with panel height, stay very small

[Greg Varsanyi via KDE Bugzilla]

https://bugs.kde.org/show_bug.cgi?id=360333

--- Comment #8 from Greg Varsanyi  ---
(In reply to Marco Martin from comment #6)
> this bug is pretty much mutually exclusive with
> https://bugs.kde.org/show_bug.cgi?id=353834
> I fear icons are either going to be too big for some people or too small for
> some other people (and i don't want to add a configure icon size ui)

This maybe an issue VDG could be involved with.

I'm not sure what you mean by "too big for some", but systray icons did indeed
look bigger than the clock font next to them (since both got the same height,
but fonts have some padding b/c of unused accent space above and below). Having
that aligned would be a win.

But now it's not following the height of the panel at all. Fixed height systray
icons mean they will look wrong on every resized taskbar panel - proportionally
with how much it is resized.

Bringing back the proportionate design with make it much more usable for many
of us, making it aligned with the text-based plasmoids on the panel would just
make it look awesome.

-- 
You are receiving this mail because:
You are watching all bug changes.