Merge lp:~pkunal-parmar/ubuntu-calendar-app/IncubteObject into lp:ubuntu-calendar-app

Proposed by Kunal Parmar
Status: Merged
Approved by: Mihir Soni
Approved revision: 539
Merged at revision: 545
Proposed branch: lp:~pkunal-parmar/ubuntu-calendar-app/IncubteObject
Merge into: lp:ubuntu-calendar-app
Diff against target: 31 lines (+15/-2)
1 file modified
TimeLineBase.qml (+15/-2)
To merge this branch: bzr merge lp:~pkunal-parmar/ubuntu-calendar-app/IncubteObject
Reviewer Review Type Date Requested Status
Mihir Soni Approve
Ubuntu Phone Apps Jenkins Bot continuous-integration Approve
Review via email: mp+241166@code.launchpad.net

Commit message

using incubate object to create bubble object

Description of the change

using incubate object to create bubble object

To post a comment you must log in.
Revision history for this message
Ubuntu Phone Apps Jenkins Bot (ubuntu-phone-apps-jenkins-bot) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
Ubuntu Phone Apps Jenkins Bot (ubuntu-phone-apps-jenkins-bot) wrote :
review: Approve (continuous-integration)
Revision history for this message
Mihir Soni (mihirsoni) wrote :

Looks good to me !!
Thanks kunal

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'TimeLineBase.qml'
--- TimeLineBase.qml 2014-11-06 13:49:22 +0000
+++ TimeLineBase.qml 2014-11-08 02:18:28 +0000
@@ -169,12 +169,25 @@
169 function createEvent( event, depth, sizeOfRow ) {169 function createEvent( event, depth, sizeOfRow ) {
170 var eventBubble;170 var eventBubble;
171 if( isHashEmpty(intern.unUsedEvents) ) {171 if( isHashEmpty(intern.unUsedEvents) ) {
172 eventBubble = delegate.createObject(bubbleOverLay);172 var incubator = delegate.incubateObject(bubbleOverLay);
173 eventBubble.objectName = children.length;173 if (incubator.status !== Component.Ready) {
174 incubator.onStatusChanged = function(status) {
175 if (status === Component.Ready) {
176 incubator.object.objectName = children.length;
177 assignBubbleProperties(incubator.object, event, depth, sizeOfRow);
178 }
179 }
180 } else {
181 incubator.object.objectName = children.length;
182 assignBubbleProperties(incubator.object, event, depth, sizeOfRow);
183 }
174 } else {184 } else {
175 eventBubble = getUnusedEventBubble();185 eventBubble = getUnusedEventBubble();
186 assignBubbleProperties(eventBubble, event, depth, sizeOfRow);
176 }187 }
188 }
177189
190 function assignBubbleProperties(eventBubble, event, depth, sizeOfRow) {
178 var hour = event.startDateTime.getHours();191 var hour = event.startDateTime.getHours();
179 var yPos = (( event.startDateTime.getMinutes() * hourHeight) / 60) + hour * hourHeight192 var yPos = (( event.startDateTime.getMinutes() * hourHeight) / 60) + hour * hourHeight
180 eventBubble.y = yPos;193 eventBubble.y = yPos;

Subscribers

People subscribed via source and target branches

to status/vote changes: