Add High DPI support
Needs ReviewPublic

Authored by volkov on Mar 20 2019, 4:38 PM.

Details

Reviewers
None
Group Reviewers
KDE Games

Diff Detail

Repository
R410 KPatience
Branch
Applications/19.04
Lint
No Linters Available
Unit
No Unit Test Coverage
Build Status
Buildable 9934
Build 9952: arc lint + arc unit
volkov created this revision.Mar 20 2019, 4:38 PM
Restricted Application added a reviewer: KDE Games. · View Herald TranscriptMar 20 2019, 4:38 PM
Restricted Application added a subscriber: kde-games-devel. · View Herald Transcript
volkov requested review of this revision.Mar 20 2019, 4:38 PM

@davidedmundson you know stuff about hidpi, can you give this a look?

apol added a subscriber: apol.Mar 20 2019, 9:39 PM

Just gave it a try, it's super tiny.

This is strange. I can't reproduce it.
Could you provide more information?

  • Wayland or X11
  • version of Qt
  • env | grep QT
libkcardgame/kcardthemewidget.cpp
85

this image doesn't have the DPR set

105

Any patch that has code to render bigger is normally a bad sign.

We should only be resizing the backing stores and having the DPR set on that backing store.

The rest will implicitly "just work".

207

why?

The image coming in should have a DPR set.

Also tiny for me.

  • Wayland or X11

X11

  • version of Qt

5.12.2

  • env | grep QT

QT_AUTO_SCREEN_SCALE_FACTOR=0
QT_PLUGIN_PATH=/usr/host/lib/qt5/plugins
QT_SCREEN_SCALE_FACTORS=HDMI-1=1;HDMI-2=1;DP-1=1;HDMI-3=1;

Fix for tiny cards: D20061