[Merge] lp:~gerboland/qtubuntu/make-up-physical-size-if-mir-says-0x0 into lp:qtubuntu
Nick Dedekind
nick.dedekind at canonical.com
Wed Nov 2 09:01:26 UTC 2016
Review: Needs Fixing
Looks ok; just one comment.
Diff comments:
> === modified file 'src/ubuntumirclient/screen.cpp'
> --- src/ubuntumirclient/screen.cpp 2016-04-28 14:09:54 +0000
> +++ src/ubuntumirclient/screen.cpp 2016-08-10 16:28:11 +0000
> @@ -155,13 +170,17 @@
>
> // Mode = Resolution & refresh rate
> const MirOutputMode *mode = mir_output_get_current_mode(output);
> - mNativeGeometry.setX(mir_output_get_position_x(output));
> - mNativeGeometry.setY(mir_output_get_position_y(output));
> - mNativeGeometry.setWidth(mir_output_mode_get_width(mode));
> - mNativeGeometry.setHeight(mir_output_mode_get_height(mode));
> + mGeometry.setX(mir_output_get_position_x(output));
> + mGeometry.setY(mir_output_get_position_y(output));
> + mGeometry.setWidth(mir_output_mode_get_width(mode));
> + mGeometry.setHeight(mir_output_mode_get_height(mode));
>
> mRefreshRate = mir_output_mode_get_refresh_rate(mode);
>
> + // Physical screen size (in mm)
> + QSize mirPhysicalSize(mir_output_get_physical_width_mm(output), mir_output_get_physical_height_mm(output));
> + mPhysicalSize = setPhysicalSize(mirPhysicalSize, mGeometry.size());
what are you setting? getPhysicalSize/fromPhysicalSize
> +
> // UI scale & DPR
> mScale = mir_output_get_scale_factor(output);
> if (overrideDevicePixelRatio > 0) {
--
https://code.launchpad.net/~gerboland/qtubuntu/make-up-physical-size-if-mir-says-0x0/+merge/302578
Your team Ubuntu Phablet Team is subscribed to branch lp:qtubuntu.
More information about the Ubuntu-reviews
mailing list