Differential D1605

[plugins/qpa] Add support for QT_WAYLAND_FORCE_DPI env variable
ClosedPublic
Actions

Authored by graesslin on May 13 2016, 8:22 AM.

Details

Reviewers
broulik
Group Reviewers
Plasma
Commits
R108:1350653d4497: [plugins/qpa] Add support for QT_WAYLAND_FORCE_DPI env variable
Summary

If a user specifies the QT_WAYLAND_FORCE_DPI env variable, KWin uses
it to force a logicalDPI, just like QtWayland.

Test Plan

Normally sized window decorations

Diff Detail

Repository
R108 KWin
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
graesslin updated this revision to Diff 3801.May 13 2016, 8:22 AM
graesslin retitled this revision from to [plugins/qpa] Add support for QT_WAYLAND_FORCE_DPI env variable.
graesslin updated this object.
graesslin edited the test plan for this revision. (Show Details)
graesslin added a reviewer: Plasma.
Restricted Application added a project: Plasma. · View Herald TranscriptMay 13 2016, 8:22 AM
Restricted Application added a subscriber: plasma-devel. · View Herald Transcript
broulik added a subscriber: broulik.May 13 2016, 8:41 AM
broulik added inline comments.
plugins/qpa/screen.cpp
67

qEnvironmentVariableIsSet and qEnvironmentVariableIntValue

68

Braces pls

graesslin added inline comments.May 13 2016, 9:09 AM
plugins/qpa/screen.cpp
68

that's what one gets for copying code from Qt ;-)

graesslin updated this revision to Diff 3803.May 13 2016, 9:14 AM
  • qEnvironmentVariableIsSet
  • coding style
  • raise Qt version for qEnvironmentVariableIntValue
broulik accepted this revision.May 13 2016, 12:37 PM
broulik added a reviewer: broulik.
This revision is now accepted and ready to land.May 13 2016, 12:37 PM
Closed by commit R108:1350653d4497: [plugins/qpa] Add support for QT_WAYLAND_FORCE_DPI env variable (authored by graesslin). · Explain WhyMay 17 2016, 5:35 AM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathPackages
MCMakeLists.txt (2 lines)
Mplugins/qpa/screen.h (1 line)
Mplugins/qpa/screen.cpp (10 lines)

Diff 3829

View Options

CMakeLists.txt

Loading...
View Options

plugins/qpa/screen.h

Loading...
View Options

plugins/qpa/screen.cpp

Loading...