Merge lp:~marcinello/ubuntu-calendar-app/fix-1297463 into lp:ubuntu-calendar-app

Proposed by Marcin
Status: Merged
Approved by: Kunal Parmar
Approved revision: 225
Merged at revision: 227
Proposed branch: lp:~marcinello/ubuntu-calendar-app/fix-1297463
Merge into: lp:ubuntu-calendar-app
Diff against target: 12 lines (+1/-1)
1 file modified
calendar.qml (+1/-1)
To merge this branch: bzr merge lp:~marcinello/ubuntu-calendar-app/fix-1297463
Reviewer Review Type Date Requested Status
Ubuntu Phone Apps Jenkins Bot continuous-integration Approve
Alan Pope 🍺🐧🐱 πŸ¦„ (community) Approve
Review via email: mp+213381@code.launchpad.net

Commit message

Simply changed default window size to match music-app, calendar-app does not use ConditionalLayout at the moment

Description of the change

Simply changed default window size to match music-app, calendar-app does not use ConditionalLayout at the moment

To post a comment you must log in.
Revision history for this message
Alan Pope 🍺🐧🐱 πŸ¦„ (popey) :
review: Approve
Revision history for this message
Kunal Parmar (pkunal-parmar) wrote :

I have few question not related to this changes but general direction.

Does SDK resizes app to phone's screen dimension when on phone ?

And Is SDK planning to provide some hint when its on desktop and when its on Phone and launch app with certain default dimension ?

Revision history for this message
Alan Pope 🍺🐧🐱 πŸ¦„ (popey) wrote :

The app appears "full size" on the phone and tablet - that is, it takes up the entire screen (indicator aside). The exception to this is on a tablet where the app is confined to the sidestage, where the application will sit in a phone-sized "window".

Revision history for this message
Ubuntu Phone Apps Jenkins Bot (ubuntu-phone-apps-jenkins-bot) :
review: Approve (continuous-integration)

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'calendar.qml'
--- calendar.qml 2014-03-26 14:39:17 +0000
+++ calendar.qml 2014-03-30 13:40:11 +0000
@@ -51,7 +51,7 @@
51 objectName: "calendar"51 objectName: "calendar"
52 applicationName: "com.ubuntu.calendar"52 applicationName: "com.ubuntu.calendar"
5353
54 width: units.gu(45)54 width: units.gu(100)
55 height: units.gu(80)55 height: units.gu(80)
5656
57 headerColor: "#266249"57 headerColor: "#266249"

Subscribers

People subscribed via source and target branches

to status/vote changes: