Merge lp:~bzoltan/ubuntu-calendar-app/lp_1351024_workaround into lp:ubuntu-calendar-app

Proposed by Zoltan Balogh
Status: Rejected
Rejected by: Nekhelesh Ramananthan
Proposed branch: lp:~bzoltan/ubuntu-calendar-app/lp_1351024_workaround
Merge into: lp:ubuntu-calendar-app
Diff against target: 12 lines (+1/-1)
1 file modified
NewEvent.qml (+1/-1)
To merge this branch: bzr merge lp:~bzoltan/ubuntu-calendar-app/lp_1351024_workaround
Reviewer Review Type Date Requested Status
Nekhelesh Ramananthan Disapprove
Review via email: mp+230095@code.launchpad.net

Commit message

Use the JS PickerPanel to work around the #1351024

Description of the change

Use the JS PickerPanel to work around the #1351024

To post a comment you must log in.
Revision history for this message
Timo Jyrinki (timo-jyrinki) wrote :

Looks identical to my branch, not working at least yet with the current UITK proposal:
https://bugs.launchpad.net/ubuntu-calendar-app/+bug/1351024/comments/20

Revision history for this message
Nekhelesh Ramananthan (nik90) wrote :

I believe we won't be needing this any longer since the date picker works as expected in calendar. I am thereby rejecting this MP.

review: Disapprove

Unmerged revisions

387. By Zoltan Balogh

Use the JS PickerPanel to work around the #1351024

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'NewEvent.qml'
--- NewEvent.qml 2014-08-01 14:28:34 +0000
+++ NewEvent.qml 2014-08-08 12:08:51 +0000
@@ -276,7 +276,7 @@
276276
277 function openDatePicker (element, caller, callerProperty, mode) {277 function openDatePicker (element, caller, callerProperty, mode) {
278 element.highlighted = true;278 element.highlighted = true;
279 var picker = PickerPanel.openDatePicker(caller, callerProperty, mode);279 var picker = PickerPanelWorkaround.openDatePicker(caller, callerProperty, mode);
280 if (!picker) return;280 if (!picker) return;
281 picker.closed.connect(function () {281 picker.closed.connect(function () {
282 element.highlighted = false;282 element.highlighted = false;

Subscribers

People subscribed via source and target branches

to status/vote changes: