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
1=== modified file 'CMakeLists.txt'
2--- CMakeLists.txt 2012-11-20 12:36:36 +0000
3+++ CMakeLists.txt 2012-12-27 16:05:27 +0000
4@@ -145,9 +145,9 @@
5 )
6
7 add_definitions(${CFLAGS} -Wall -Winit-self -Wwrite-strings -Wunreachable-code -Wstrict-prototypes )
8-add_executable(maya ${VALA_C})
9+add_executable(maya-calendar ${VALA_C})
10
11-install(TARGETS maya RUNTIME DESTINATION bin)
12+install(TARGETS maya-calendar RUNTIME DESTINATION bin)
13
14 #
15 # GSettings Schema
16
17=== modified file 'Maya/Application.vala'
18--- Maya/Application.vala 2012-11-23 17:22:11 +0000
19+++ Maya/Application.vala 2012-12-27 16:05:27 +0000
20@@ -284,7 +284,9 @@
21 void on_modified (E.CalComponent comp) {
22 var dialog = new Maya.View.EventDialog (window, sourcemgr, comp, comp.get_data<E.Source>("source"), false);
23 dialog.response.connect ((response_id) => on_event_dialog_response(dialog, response_id, false));
24+ dialog.load();
25 dialog.present ();
26+
27 }
28
29 /**
30
31=== modified file 'Maya/View/EventDialog.vala'
32--- Maya/View/EventDialog.vala 2012-09-16 17:10:50 +0000
33+++ Maya/View/EventDialog.vala 2012-12-27 16:05:27 +0000
34@@ -112,7 +112,7 @@
35 build_dialog (add_event);
36
37 // Load the event's properties in to the dialog
38- load ();
39+ // load ();
40
41 update_create_sensitivity ();
42 }
43@@ -215,7 +215,7 @@
44 /**
45 * Populate the dialog's widgets with the component's values.
46 */
47- void load () {
48+ public void load () {
49
50 unowned iCal.icalcomponent comp = ecal.get_icalcomponent ();
51
52@@ -322,9 +322,15 @@
53
54 to_date_picker = make_date_picker ();
55 to_date_picker.notify["date"].connect ( () => {on_date_modified(1);} );
56+
57+
58+ DateTime from_time = new DateTime.now_local();
59+ DateTime to_time = new DateTime.now_local();
60+
61 to_time_picker = make_time_picker ();
62 to_time_picker.time_changed.connect ( () => {on_time_modified(1);} );
63
64+
65 allday_switch_grid.attach (allday_switch, 0, 0, 1, 1);
66
67 allday_switch_grid.set_valign (Gtk.Align.CENTER);
68
69=== modified file 'data/maya.desktop'
70--- data/maya.desktop 2012-11-09 21:43:42 +0000
71+++ data/maya.desktop 2012-12-27 16:05:27 +0000
72@@ -4,7 +4,7 @@
73 Name=Maya
74 GenericName=Calendar
75 Comment=View and schedule events
76-Exec=maya
77+Exec=maya-calendar
78 Icon=office-calendar
79 Terminal=false
80 Categories=GTK;Office;Calendar;
81@@ -16,5 +16,5 @@
82
83 [AddEvent Shortcut Group]
84 Name=_Add Event
85-Exec=maya -a
86+Exec=maya-calendar -a
87 TargetEnvironment=Unity

Subscribers

People subscribed via source and target branches