Ubuntu
ubuntu-ui-toolkit package

Base Grid units on the gsettings key per monitor

Bug #1306581 reported by Pat McGowan
10
This bug affects 2 people
Affects Status Importance Assigned to Milestone
apparmor-easyprof-ubuntu (Ubuntu)
Confirmed
High
Unassigned
ubuntu-ui-toolkit (Ubuntu)
Confirmed
High
Zsombor Egri

Bug Description

On the desktop the gsettings parameter 'com.ubuntu.user-interface scale-factor' key contains the values for grid units for each monitor on the system. This setting should also adjust the touch apps on the desktop built with the toolkit.

As a first step it would be good to have the built in monitor adjust properly as that is likely to be the touch enabled screen on the laptop. Secondarily then we would dynamically adjust the app when running on the external monitors.

For example {'HDMI1': 8, 'eDP1': 15} tells you the built-in display (eDPI) has about 15 pixels per grid unit and the external display is using the default of 8 pixels per grid unit.

Zsombor Egri (zsombi)
Changed in ubuntu-ui-toolkit:
milestone: none → june-2014
importance: High → Critical
Revision history for this message
Cris Dywan (kalikiana) wrote :

FYI Screen has a name property that corresponds to the display name on X11:
http://qt-project.org/doc/qt-5/qscreen.html#name-prop
http://qt-project.org/doc/qt-5/qml-qtquick-window-screen.html#name-prop

Revision history for this message
Zsombor Egri (zsombi) wrote :

We need this in C++. We can make a dependency towards qtgsettings and read the key value for the current monitor whenever GRID_UNIT_PX is not specified. Accessing gsettings on phone is blocked by app rights, so we should skip that for now.

Florian's DPR patch would be needed to get the DPI per screen, however with X11we may get it done using QQuickScreen code.

Pat McGowan (pat-mcgowan)
Changed in ubuntu-ui-toolkit:
status: New → In Progress
Zsombor Egri (zsombi)
Changed in ubuntu-ui-toolkit:
milestone: june-2014 → august-2014
Zsombor Egri (zsombi)
Changed in ubuntu-ui-toolkit:
status: In Progress → Confirmed
Zoltan Balogh (bzoltan)
Changed in ubuntu-ui-toolkit:
milestone: august-2014 → 10.24
Zoltan Balogh (bzoltan)
Changed in ubuntu-ui-toolkit:
milestone: 10.24 → 11.07
importance: Critical → High
Zsombor Egri (zsombi)
Changed in ubuntu-ui-toolkit:
milestone: 11.07 → 11.21
Revision history for this message
Zsombor Egri (zsombi) wrote :

A note for myself to attach the bug to indicator-display-service to provide a dubs interface to read the screen info.

Revision history for this message
Jamie Strandboge (jdstrand) wrote :

Adding an apparmor-easyprof-ubuntu task since policy will need to be updated to allow confined apps to talk to this service. It is advisable that the person implementing this DBus API talk with the security team to make sure the chosen path will work without our confinement strategy.

Changed in apparmor-easyprof-ubuntu (Ubuntu):
status: New → Confirmed
importance: Undecided → High
tags: added: application-confinement
Zoltan Balogh (bzoltan)
Changed in ubuntu-ui-toolkit (Ubuntu):
assignee: nobody → Zsombor Egri (zsombi)
importance: Undecided → High
status: New → Confirmed
Zoltan Balogh (bzoltan)
no longer affects: ubuntu-ui-toolkit
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.
Subscribing...

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.