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 wrote in panelview.cpp:1096 > 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 having the old thickness/setThickness etc that still accept sizes in > pixels for compatibility and have new setUnitsThickness or whatever that > accepts units? I can, but that just moves the conversion into being in multiple places instead of just 1. None of the other users of the thickness (the drag handles or working out the workspace screen area) want grid units. > mart wrote in panelview.cpp:1098 > in the scripting the m height is binded, not the width you're right. int gridUnit = QFontMetrics(QGuiApplication::font()).boundingRect(QStringLiteral("M")).height(); Will fix. REPOSITORY rPLASMAWORKSPACE Plasma Workspace REVISION DETAIL https://phabricator.kde.org/D1986 EMAIL PREFERENCES https://phabricator.kde.org/settings/panel/emailpreferences/ To: davidedmundson, #plasma Cc: mart, plasma-devel, jensreuterberg, sebas
_______________________________________________ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel