Merge lp:~b-beelitz/maya/fix-1080349 into lp:~elementary-apps/maya/trunk

Proposed by Benjamin Beelitz
Status: Rejected
Rejected by: David Gomes
Proposed branch: lp:~b-beelitz/maya/fix-1080349
Merge into: lp:~elementary-apps/maya/trunk
Diff against target: 87 lines (+14/-6)
4 files modified
CMakeLists.txt (+2/-2)
Maya/Application.vala (+2/-0)
Maya/View/EventDialog.vala (+8/-2)
data/maya.desktop (+2/-2)
To merge this branch: bzr merge lp:~b-beelitz/maya/fix-1080349
Reviewer Review Type Date Requested Status
David Gomes (community) Needs Resubmitting
Review via email: mp+141369@code.launchpad.net

Description of the change

The add popover event now always have the actual time of the PC. The Edit Dialog is not affected.

To post a comment you must log in.
Revision history for this message
Raphael Isemann (teemperor) wrote :

That review looks like it is already got merged? At least that "actual" time of the PC thing and so on. Can you give me a feedback if this merge request is still necessary?

Revision history for this message
Benjamin Beelitz (b-beelitz) wrote :

Hi

I don't know. You mean the fix that maya has been renamed? I don´t know if
my fix is merged and necessary. It was an bug. I don´t know if there is
still an bug. Sorry.

Benjamin
Am 02.05.2013 18:28 schrieb "Raphael Isemann" <email address hidden>:

> That review looks like it is already got merged? At least that "actual"
> time of the PC thing and so on. Can you give me a feedback if this merge
> request is still necessary?
> --
> https://code.launchpad.net/~b-beelitz/maya/fix-1080349/+merge/141369
> You are the owner of lp:~b-beelitz/maya/fix-1080349.
>

Revision history for this message
David Gomes (davidgomes) wrote :

>2012-12-27
December of 2012 was just too long ago.

review: Needs Resubmitting

Unmerged revisions

391. By Benjamin Beelitz

The add popover event now always have the actual time of the PC. The edit PopOver has the Event-Time and so on. It's pretty cool know

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'CMakeLists.txt'
--- CMakeLists.txt 2012-11-20 12:36:36 +0000
+++ CMakeLists.txt 2012-12-27 16:05:27 +0000
@@ -145,9 +145,9 @@
145 )145 )
146146
147add_definitions(${CFLAGS} -Wall -Winit-self -Wwrite-strings -Wunreachable-code -Wstrict-prototypes )147add_definitions(${CFLAGS} -Wall -Winit-self -Wwrite-strings -Wunreachable-code -Wstrict-prototypes )
148add_executable(maya ${VALA_C})148add_executable(maya-calendar ${VALA_C})
149149
150install(TARGETS maya RUNTIME DESTINATION bin)150install(TARGETS maya-calendar RUNTIME DESTINATION bin)
151151
152#152#
153# GSettings Schema153# GSettings Schema
154154
=== modified file 'Maya/Application.vala'
--- Maya/Application.vala 2012-11-23 17:22:11 +0000
+++ Maya/Application.vala 2012-12-27 16:05:27 +0000
@@ -284,7 +284,9 @@
284 void on_modified (E.CalComponent comp) {284 void on_modified (E.CalComponent comp) {
285 var dialog = new Maya.View.EventDialog (window, sourcemgr, comp, comp.get_data<E.Source>("source"), false);285 var dialog = new Maya.View.EventDialog (window, sourcemgr, comp, comp.get_data<E.Source>("source"), false);
286 dialog.response.connect ((response_id) => on_event_dialog_response(dialog, response_id, false));286 dialog.response.connect ((response_id) => on_event_dialog_response(dialog, response_id, false));
287 dialog.load();
287 dialog.present ();288 dialog.present ();
289
288 }290 }
289291
290 /**292 /**
291293
=== modified file 'Maya/View/EventDialog.vala'
--- Maya/View/EventDialog.vala 2012-09-16 17:10:50 +0000
+++ Maya/View/EventDialog.vala 2012-12-27 16:05:27 +0000
@@ -112,7 +112,7 @@
112 build_dialog (add_event);112 build_dialog (add_event);
113113
114 // Load the event's properties in to the dialog114 // Load the event's properties in to the dialog
115 load ();115 // load ();
116116
117 update_create_sensitivity ();117 update_create_sensitivity ();
118 }118 }
@@ -215,7 +215,7 @@
215 /**215 /**
216 * Populate the dialog's widgets with the component's values.216 * Populate the dialog's widgets with the component's values.
217 */217 */
218 void load () {218 public void load () {
219219
220 unowned iCal.icalcomponent comp = ecal.get_icalcomponent ();220 unowned iCal.icalcomponent comp = ecal.get_icalcomponent ();
221221
@@ -322,9 +322,15 @@
322322
323 to_date_picker = make_date_picker ();323 to_date_picker = make_date_picker ();
324 to_date_picker.notify["date"].connect ( () => {on_date_modified(1);} );324 to_date_picker.notify["date"].connect ( () => {on_date_modified(1);} );
325
326
327 DateTime from_time = new DateTime.now_local();
328 DateTime to_time = new DateTime.now_local();
329
325 to_time_picker = make_time_picker ();330 to_time_picker = make_time_picker ();
326 to_time_picker.time_changed.connect ( () => {on_time_modified(1);} );331 to_time_picker.time_changed.connect ( () => {on_time_modified(1);} );
327332
333
328 allday_switch_grid.attach (allday_switch, 0, 0, 1, 1);334 allday_switch_grid.attach (allday_switch, 0, 0, 1, 1);
329335
330 allday_switch_grid.set_valign (Gtk.Align.CENTER);336 allday_switch_grid.set_valign (Gtk.Align.CENTER);
331337
=== modified file 'data/maya.desktop'
--- data/maya.desktop 2012-11-09 21:43:42 +0000
+++ data/maya.desktop 2012-12-27 16:05:27 +0000
@@ -4,7 +4,7 @@
4Name=Maya4Name=Maya
5GenericName=Calendar5GenericName=Calendar
6Comment=View and schedule events6Comment=View and schedule events
7Exec=maya7Exec=maya-calendar
8Icon=office-calendar8Icon=office-calendar
9Terminal=false9Terminal=false
10Categories=GTK;Office;Calendar;10Categories=GTK;Office;Calendar;
@@ -16,5 +16,5 @@
1616
17[AddEvent Shortcut Group]17[AddEvent Shortcut Group]
18Name=_Add Event18Name=_Add Event
19Exec=maya -a19Exec=maya-calendar -a
20TargetEnvironment=Unity20TargetEnvironment=Unity

Subscribers

People subscribed via source and target branches