Differential D13309

Enable high DPI support
Needs ReviewPublic
Actions

Authored by kross on Jun 3 2018, 6:51 PM.

Details

Reviewers
mutlaqja
Summary

This was originally disabled because the version of Qt that KStars was using was older than the version of Qt that KStars Lite was using, and didn't have High DPI support. KStars is now using a much more recent version of Qt, and has full High DPI support. So let's use it!

This makes things look much better on my 4K monitor. Previously some things would scale, and some things wouldn't, making things look a little weird. Now everything scales properly.

Diff Detail

Repository
R321 KStars
Lint
Lint Skipped
Unit
Unit Tests Skipped
kross created this revision.Jun 3 2018, 6:51 PM
Restricted Application added a project: KDE Edu. · View Herald TranscriptJun 3 2018, 6:51 PM
Restricted Application added a subscriber: kde-edu. · View Herald Transcript
kross requested review of this revision.Jun 3 2018, 6:51 PM
apol added a subscriber: apol.EditedJun 5 2018, 5:29 PM

+1 makes a lot of sense. Only reason not to if there were components not adapted to rendering on high dpi, but then they will break on Android.

mutlaqja added a comment.Jun 6 2018, 6:14 AM

What version of Qt was this officially supported?

mutlaqja added a comment.Jun 6 2018, 6:46 AM

Ok since Qt 5.6, so I'll enable it for Qt 5.6 and above.

mutlaqja resigned from this revision.Jun 6 2018, 6:49 AM

Revision Contents
Changeset List

PathPackages
Mkstars/main.cpp (2 lines)

Diff 35475

View Options

kstars/main.cpp

Loading...