davidedmundson abandoned this revision.
davidedmundson added a comment.
Can't remember what the state of this is, and the original code probably
changed. Closing.
REPOSITORY
R120 Plasma Workspace
REVISION DETAIL
https://phabricator.kde.org/D1986
To: davidedmundson, #plasma
Cc: mart, pla
davidedmundson added a comment.
> uh, also no more plans of supporting QT_SCREEN_SCALE_FACTORS instead? (in
this case the pixel based sizes should be just fine as they would be scaled?)
We /could/ but using a hybrid of two scaling systems seems like it'll get
complex
INLINE COMMENTS
>
mart added a comment.
anyways, i like the idea.
uh, also no more plans of supporting QT_SCREEN_SCALE_FACTORS instead? (in
this case the pixel based sizes should be just fine as they would be scaled?)
REPOSITORY
rPLASMAWORKSPACE Plasma Workspace
REVISION DETAIL
https://phabricator.kd
mart added inline comments.
INLINE COMMENTS
> panelview.cpp:1096
>
> +void PanelView::writeScaledConfigEntry(const char *name, int pixelSize)
> +{
why adding a new setter and getter that work in pixels? if something like that
should be added, i wuld like just working with grid units.
perhaps
davidedmundson created this revision.
davidedmundson added a reviewer: Plasma.
Restricted Application added a project: Plasma.
Restricted Application added a subscriber: plasma-devel.
REVISION SUMMARY
This is useful for HighDPI otherwise a user needs to manually resize the
panel initially and/