D14697: X11: set GDK_SCALE (try 2)

2018-10-18 Thread Nathaniel Graham
This revision was not accepted when it landed; it landed in state "Needs Revision". This revision was automatically updated to reflect the committed changes. Closed by commit R120:31e1a1a96ca2: X11: set GDK_SCALE (try 2) (authored by eszlari, committed by ngraham). REPOSITORY R120 Plasma Works

D14697: X11: set GDK_SCALE (try 2)

2018-10-18 Thread Peter Eszlari
eszlari added a comment. peter.eszl...@gmail.com REPOSITORY R120 Plasma Workspace REVISION DETAIL https://phabricator.kde.org/D14697 To: eszlari, davidedmundson, ngraham, #plasma Cc: broulik, plasma-devel, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas,

D14697: X11: set GDK_SCALE (try 2)

2018-10-18 Thread Nathaniel Graham
ngraham added a comment. @eszlari, can you provide an email address so we can land this with the proper authorship information? REPOSITORY R120 Plasma Workspace REVISION DETAIL https://phabricator.kde.org/D14697 To: eszlari, davidedmundson, ngraham, #plasma Cc: broulik, plasma-devel, ra

D14697: X11: set GDK_SCALE (try 2)

2018-10-18 Thread Nathaniel Graham
ngraham added a comment. Peter, can you provide an email address so we can land this with the proper authorship information? REPOSITORY R120 Plasma Workspace REVISION DETAIL https://phabricator.kde.org/D14697 To: eszlari, davidedmundson, ngraham, #plasma Cc: broulik, plasma-devel, ragre

D14697: X11: set GDK_SCALE (try 2)

2018-10-18 Thread David Edmundson
davidedmundson added a comment. sure. REPOSITORY R120 Plasma Workspace REVISION DETAIL https://phabricator.kde.org/D14697 To: eszlari, davidedmundson, ngraham, #plasma Cc: broulik, plasma-devel, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart

D14697: X11: set GDK_SCALE (try 2)

2018-10-18 Thread Nathaniel Graham
ngraham added a comment. I think that would make sense. @davidedmundson do you agree? It would be nice to have three months to live with this and see if there are any bugs (my testing is clean FWIW). REPOSITORY R120 Plasma Workspace REVISION DETAIL https://phabricator.kde.org/D14697 To

D14697: X11: set GDK_SCALE (try 2)

2018-10-18 Thread Kai Uwe Broulik
broulik resigned from this revision. broulik added a comment. I guess this can go in so we can have a full cycle of testing? REPOSITORY R120 Plasma Workspace REVISION DETAIL https://phabricator.kde.org/D14697 To: eszlari, davidedmundson, ngraham, #plasma Cc: broulik, plasma-devel, ragree

D14697: X11: set GDK_SCALE (try 2)

2018-10-09 Thread David Edmundson
davidedmundson added inline comments. INLINE COMMENTS > broulik wrote in startkde.cmake:98 > Where does `kdeglobals_kscreen_scalefactor` come from? kstartupconfig5, which reads startupconfigkeys to see what to extract. They then get set as env vars. REPOSITORY R120 Plasma Workspace REVISION

D14697: X11: set GDK_SCALE (try 2)

2018-10-09 Thread Peter Eszlari
eszlari added inline comments. INLINE COMMENTS > broulik wrote in startkde.cmake:98 > Where does `kdeglobals_kscreen_scalefactor` come from? Line 52. REPOSITORY R120 Plasma Workspace REVISION DETAIL https://phabricator.kde.org/D14697 To: eszlari, davidedmundson, ngraham, #plasma, broulik

D14697: X11: set GDK_SCALE (try 2)

2018-10-09 Thread Kai Uwe Broulik
broulik added inline comments. INLINE COMMENTS > startkde.cmake:98 > export QT_SCREEN_SCALE_FACTORS="$kdeglobals_kscreen_screenscalefactors" > +if [ "$kdeglobals_kscreen_scalefactor" -eq "2" ] || [ > "$kdeglobals_kscreen_scalefactor" -eq "3" ]; then > +export GDK_SCALE=$kdegloba

D14697: X11: set GDK_SCALE (try 2)

2018-10-09 Thread Nathaniel Graham
ngraham added a comment. @davidedmundson 5.14 was just released; should we give this a shot for 5.15? REPOSITORY R120 Plasma Workspace REVISION DETAIL https://phabricator.kde.org/D14697 To: eszlari, davidedmundson, ngraham, #plasma, broulik Cc: broulik, plasma-devel, ragreen, Pitel, Zren

D14697: X11: set GDK_SCALE (try 2)

2018-08-20 Thread Nathaniel Graham
ngraham added a comment. It works well enough for me so far. Can we get some more testing? REPOSITORY R120 Plasma Workspace REVISION DETAIL https://phabricator.kde.org/D14697 To: eszlari, davidedmundson, ngraham, #plasma, broulik Cc: broulik, plasma-devel, ragreen, Pitel, ZrenBot, leslie

D14697: X11: set GDK_SCALE (try 2)

2018-08-13 Thread Kai Uwe Broulik
broulik requested changes to this revision. broulik added a comment. This revision now requires changes to proceed. Please hang on, we need to more thorougly test this. REPOSITORY R120 Plasma Workspace REVISION DETAIL https://phabricator.kde.org/D14697 To: eszlari, davidedmundson, ngraha

D14697: X11: set GDK_SCALE (try 2)

2018-08-13 Thread Nathaniel Graham
ngraham accepted this revision. ngraham added a reviewer: Plasma. ngraham added a comment. This revision is now accepted and ready to land. Tested this out and it works flawlessly for Firefox, Lollypop, and GIMP (to the extent that GIMP does support HiDPI) with the integer scale factor use c

D14697: X11: set GDK_SCALE (try 2)

2018-08-08 Thread Peter Eszlari
eszlari created this revision. Restricted Application added a project: Plasma. Restricted Application added a subscriber: plasma-devel. eszlari requested review of this revision. REVISION SUMMARY BUG: 387849 REPOSITORY R120 Plasma Workspace REVISION DETAIL https://phabricator.kde.org/D1469