Merge lp:~vikoadi/pantheon-photos/clean-menubar into lp:~pantheon-photos/pantheon-photos/trunk
- clean-menubar
- Merge into trunk
Proposed by
Viko Adi Rahmawan
Status: | Merged | ||||||||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
Approved by: | Danielle Foré | ||||||||||||||||
Approved revision: | 2604 | ||||||||||||||||
Merged at revision: | 2605 | ||||||||||||||||
Proposed branch: | lp:~vikoadi/pantheon-photos/clean-menubar | ||||||||||||||||
Merge into: | lp:~pantheon-photos/pantheon-photos/trunk | ||||||||||||||||
Diff against target: |
396 lines (+16/-119) 15 files modified
src/CollectionPage.vala (+3/-12) src/MediaPage.vala (+0/-7) src/db/PhotoTable.vala (+3/-4) ui/collection.ui (+3/-0) ui/direct.ui (+0/-32) ui/event.ui (+1/-5) ui/events_directory.ui (+1/-8) ui/import.ui (+0/-6) ui/import_queue.ui (+1/-4) ui/media.ui (+0/-11) ui/offline.ui (+0/-5) ui/photo.ui (+0/-2) ui/photo_context.ui (+1/-0) ui/tags.ui (+1/-10) ui/trash.ui (+2/-13) |
||||||||||||||||
To merge this branch: | bzr merge lp:~vikoadi/pantheon-photos/clean-menubar | ||||||||||||||||
Related bugs: |
|
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Danielle Foré | ux | Approve | |
meese | Approve | ||
Review via email: mp+232400@code.launchpad.net |
Commit message
* Move Duplicate and Remove from Library to context menu
* remove Play Video
* clean duplicated menu item
* fix photo duplicating
Description of the change
* Move Duplicate and Remove from Library to context menu
* remove Play Video
* clean duplicated menu item
* fix photo duplicating
To post a comment you must log in.
Revision history for this message
Danielle Foré (danrabbit) wrote : | # |
- 2603. By Viko Adi Rahmawan
-
fix missing popup menu, rearrange remove from library
- 2604. By Viko Adi Rahmawan
-
rearrange removetagsfromphoto
Revision history for this message
Viko Adi Rahmawan (vikoadi) wrote : | # |
I'm not really sure where should i put "Duplicate" and the arrangement of Remove from Library-Move to Trash-Remove Tag from
Revision history for this message
meese (meese) wrote : | # |
Works now and code looks fine still needs a UX approve though I think?
(and sorry for breaking duplicate my bad thank you for fixing!)
review:
Approve
Revision history for this message
Danielle Foré (danrabbit) wrote : | # |
I want to sit down and come up with a good order for that menu, but let's do it in another merge once its contents settle down a bit :)
review:
Approve
(ux)
Preview Diff
[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1 | === modified file 'src/CollectionPage.vala' | |||
2 | --- src/CollectionPage.vala 2014-08-25 06:04:50 +0000 | |||
3 | +++ src/CollectionPage.vala 2014-08-28 02:36:47 +0000 | |||
4 | @@ -91,8 +91,8 @@ | |||
5 | 91 | return group; | 91 | return group; |
6 | 92 | } | 92 | } |
7 | 93 | 93 | ||
10 | 94 | private static InjectionGroup create_edit_menu_injectables () { | 94 | private static InjectionGroup create_context_menu_injectables () { |
11 | 95 | InjectionGroup group = new InjectionGroup ("/MenuBar/EditMenu/EditExtrasPlaceholder"); | 95 | InjectionGroup group = new InjectionGroup ("/CollectionContextMenu/EditExtrasPlaceholder"); |
12 | 96 | 96 | ||
13 | 97 | group.add_menu_item ("Duplicate"); | 97 | group.add_menu_item ("Duplicate"); |
14 | 98 | 98 | ||
15 | @@ -107,14 +107,6 @@ | |||
16 | 107 | return group; | 107 | return group; |
17 | 108 | } | 108 | } |
18 | 109 | 109 | ||
19 | 110 | private static InjectionGroup create_photos_menu_externals_injectables () { | ||
20 | 111 | InjectionGroup group = new InjectionGroup ("/MenuBar/PhotosMenu/PhotosExtrasExternalsPlaceholder"); | ||
21 | 112 | |||
22 | 113 | group.add_menu_item ("PlayVideo"); | ||
23 | 114 | |||
24 | 115 | return group; | ||
25 | 116 | } | ||
26 | 117 | |||
27 | 118 | protected override void init_collect_ui_filenames (Gee.List<string> ui_filenames) { | 110 | protected override void init_collect_ui_filenames (Gee.List<string> ui_filenames) { |
28 | 119 | base.init_collect_ui_filenames (ui_filenames); | 111 | base.init_collect_ui_filenames (ui_filenames); |
29 | 120 | 112 | ||
30 | @@ -227,9 +219,8 @@ | |||
31 | 227 | InjectionGroup[] groups = base.init_collect_injection_groups (); | 219 | InjectionGroup[] groups = base.init_collect_injection_groups (); |
32 | 228 | 220 | ||
33 | 229 | groups += create_file_menu_injectables (); | 221 | groups += create_file_menu_injectables (); |
35 | 230 | groups += create_edit_menu_injectables (); | 222 | groups += create_context_menu_injectables (); |
36 | 231 | groups += create_view_menu_fullscreen_injectables (); | 223 | groups += create_view_menu_fullscreen_injectables (); |
37 | 232 | groups += create_photos_menu_externals_injectables (); | ||
38 | 233 | 224 | ||
39 | 234 | return groups; | 225 | return groups; |
40 | 235 | } | 226 | } |
41 | 236 | 227 | ||
42 | === modified file 'src/MediaPage.vala' | |||
43 | --- src/MediaPage.vala 2014-08-25 06:10:33 +0000 | |||
44 | +++ src/MediaPage.vala 2014-08-28 02:36:47 +0000 | |||
45 | @@ -377,13 +377,6 @@ | |||
46 | 377 | filter_photos.label = Resources.FILTER_PHOTOS_MENU; | 377 | filter_photos.label = Resources.FILTER_PHOTOS_MENU; |
47 | 378 | actions += filter_photos; | 378 | actions += filter_photos; |
48 | 379 | 379 | ||
49 | 380 | Gtk.ActionEntry play = { "PlayVideo", Gtk.Stock.MEDIA_PLAY, TRANSLATABLE, "<Ctrl>Y", | ||
50 | 381 | TRANSLATABLE, on_play_video | ||
51 | 382 | }; | ||
52 | 383 | play.label = _ ("_Play Video"); | ||
53 | 384 | play.tooltip = _ ("Open the selected videos in the system video player"); | ||
54 | 385 | actions += play; | ||
55 | 386 | |||
56 | 387 | Gtk.ActionEntry raw_developer = { "RawDeveloper", null, TRANSLATABLE, null, null, null }; | 380 | Gtk.ActionEntry raw_developer = { "RawDeveloper", null, TRANSLATABLE, null, null, null }; |
57 | 388 | raw_developer.label = _ ("_Developer"); | 381 | raw_developer.label = _ ("_Developer"); |
58 | 389 | actions += raw_developer; | 382 | actions += raw_developer; |
59 | 390 | 383 | ||
60 | === modified file 'src/db/PhotoTable.vala' | |||
61 | --- src/db/PhotoTable.vala 2014-08-22 08:57:09 +0000 | |||
62 | +++ src/db/PhotoTable.vala 2014-08-28 02:36:47 +0000 | |||
63 | @@ -360,7 +360,7 @@ | |||
64 | 360 | + "original_orientation, import_id, event_id, transformations, md5, thumbnail_md5, " | 360 | + "original_orientation, import_id, event_id, transformations, md5, thumbnail_md5, " |
65 | 361 | + "exif_md5, time_created, flags, rating, file_format, title, backlinks, " | 361 | + "exif_md5, time_created, flags, rating, file_format, title, backlinks, " |
66 | 362 | + "time_reimported, editable_id, metadata_dirty, developer, develop_shotwell_id, " | 362 | + "time_reimported, editable_id, metadata_dirty, developer, develop_shotwell_id, " |
68 | 363 | + "develop_camera_id, develop_embedded_id, comment, enhanced, original_transforms" | 363 | + "develop_camera_id, develop_embedded_id, comment, enhanced, original_transforms " |
69 | 364 | + "FROM PhotoTable WHERE id=?", | 364 | + "FROM PhotoTable WHERE id=?", |
70 | 365 | -1, out stmt); | 365 | -1, out stmt); |
71 | 366 | assert (res == Sqlite.OK); | 366 | assert (res == Sqlite.OK); |
72 | @@ -474,8 +474,8 @@ | |||
73 | 474 | + "timestamp, exposure_time, orientation, original_orientation, import_id, event_id, " | 474 | + "timestamp, exposure_time, orientation, original_orientation, import_id, event_id, " |
74 | 475 | + "transformations, md5, thumbnail_md5, exif_md5, time_created, flags, rating, " | 475 | + "transformations, md5, thumbnail_md5, exif_md5, time_created, flags, rating, " |
75 | 476 | + "file_format, title, editable_id, developer, develop_shotwell_id, develop_camera_id, " | 476 | + "file_format, title, editable_id, developer, develop_shotwell_id, develop_camera_id, " |
78 | 477 | + "develop_embedded_id, comment, enhanced) " | 477 | + "develop_embedded_id, comment, enhanced, original_transforms) " |
79 | 478 | + "VALUES (?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?)", | 478 | + "VALUES (?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?)", |
80 | 479 | -1, out stmt); | 479 | -1, out stmt); |
81 | 480 | assert (res == Sqlite.OK); | 480 | assert (res == Sqlite.OK); |
82 | 481 | 481 | ||
83 | @@ -1283,4 +1283,3 @@ | |||
84 | 1283 | update_int64_by_id_2 (id.id, "timestamp", timestamp); | 1283 | update_int64_by_id_2 (id.id, "timestamp", timestamp); |
85 | 1284 | } | 1284 | } |
86 | 1285 | } | 1285 | } |
87 | 1286 | |||
88 | 1287 | 1286 | ||
89 | === modified file 'ui/collection.ui' | |||
90 | --- ui/collection.ui 2014-08-25 06:04:50 +0000 | |||
91 | +++ ui/collection.ui 2014-08-28 02:36:47 +0000 | |||
92 | @@ -29,6 +29,7 @@ | |||
93 | 29 | </menu> | 29 | </menu> |
94 | 30 | <separator /> | 30 | <separator /> |
95 | 31 | <menuitem name="AdjustDateTime" action="AdjustDateTime" /> | 31 | <menuitem name="AdjustDateTime" action="AdjustDateTime" /> |
96 | 32 | <placeholder name="EditExtrasPlaceholder" /> | ||
97 | 32 | <menu name="OpenWith" action="OpenWith"> | 33 | <menu name="OpenWith" action="OpenWith"> |
98 | 33 | </menu> | 34 | </menu> |
99 | 34 | <menu name="OpenWithRaw" action="OpenWithRaw"> | 35 | <menu name="OpenWithRaw" action="OpenWithRaw"> |
100 | @@ -41,6 +42,8 @@ | |||
101 | 41 | <menuitem name="ContextExport" action="Export" /> | 42 | <menuitem name="ContextExport" action="Export" /> |
102 | 42 | <placeholder name="ContractorPlaceholder" /> | 43 | <placeholder name="ContractorPlaceholder" /> |
103 | 43 | <separator /> | 44 | <separator /> |
104 | 45 | <placeholder name="ContextTagsPlaceholder" /> | ||
105 | 46 | <menuitem name="RemoveFromLibrary" action="RemoveFromLibrary" /> | ||
106 | 44 | <menuitem name="ContextMoveToTrash" action="MoveToTrash" /> | 47 | <menuitem name="ContextMoveToTrash" action="MoveToTrash" /> |
107 | 45 | </popup> | 48 | </popup> |
108 | 46 | 49 | ||
109 | 47 | 50 | ||
110 | === modified file 'ui/direct.ui' | |||
111 | --- ui/direct.ui 2014-08-25 01:57:28 +0000 | |||
112 | +++ ui/direct.ui 2014-08-28 02:36:47 +0000 | |||
113 | @@ -4,10 +4,6 @@ | |||
114 | 4 | <menu name="FileMenu" action="FileMenu"> | 4 | <menu name="FileMenu" action="FileMenu"> |
115 | 5 | <menuitem name="Save" action="Save" /> | 5 | <menuitem name="Save" action="Save" /> |
116 | 6 | <menuitem name="SaveAs" action="SaveAs" /> | 6 | <menuitem name="SaveAs" action="SaveAs" /> |
117 | 7 | <separator /> | ||
118 | 8 | <menuitem name="JumpToFile" action="CommonJumpToFile" /> | ||
119 | 9 | <separator /> | ||
120 | 10 | <menuitem name="Quit" action="CommonQuit" /> | ||
121 | 11 | </menu> | 7 | </menu> |
122 | 12 | 8 | ||
123 | 13 | <menu name="EditMenu" action="EditMenu"> | 9 | <menu name="EditMenu" action="EditMenu"> |
124 | @@ -15,34 +11,6 @@ | |||
125 | 15 | <menuitem name="Redo" action="CommonRedo" /> | 11 | <menuitem name="Redo" action="CommonRedo" /> |
126 | 16 | </menu> | 12 | </menu> |
127 | 17 | 13 | ||
128 | 18 | <menu name="ViewMenu" action="ViewMenu"> | ||
129 | 19 | <menuitem name="Fullscreen" action="CommonFullscreen" /> | ||
130 | 20 | <separator /> | ||
131 | 21 | <menuitem name="IncreaseSize" action="IncreaseSize" /> | ||
132 | 22 | <menuitem name="DecreaseSize" action="DecreaseSize" /> | ||
133 | 23 | <separator /> | ||
134 | 24 | <menuitem name="ZoomFit" action="ZoomFit" /> | ||
135 | 25 | <menuitem name="Zoom100" action="Zoom100" /> | ||
136 | 26 | <menuitem name="Zoom200" action="Zoom200" /> | ||
137 | 27 | </menu> | ||
138 | 28 | |||
139 | 29 | <menu name="PhotoMenu" action="PhotoMenu"> | ||
140 | 30 | <menuitem name="PrevPhoto" action="PrevPhoto" /> | ||
141 | 31 | <menuitem name="NextPhoto" action="NextPhoto" /> | ||
142 | 32 | <separator /> | ||
143 | 33 | <menuitem name="RotateClockwise" action="RotateClockwise" /> | ||
144 | 34 | <menuitem name="RotateCounterclockwise" action="RotateCounterclockwise" /> | ||
145 | 35 | <separator /> | ||
146 | 36 | <menu name="Tools" action="Tools"> | ||
147 | 37 | <menuitem name="Enhance" action="Enhance" /> | ||
148 | 38 | <menuitem name="Crop" action="Crop" /> | ||
149 | 39 | <menuitem name="Straighten" action="Straighten" /> | ||
150 | 40 | <menuitem name="RedEye" action="RedEye" /> | ||
151 | 41 | <menuitem name="Adjust" action="Adjust" /> | ||
152 | 42 | </menu> | ||
153 | 43 | <menuitem name="Revert" action="Revert" /> | ||
154 | 44 | </menu> | ||
155 | 45 | |||
156 | 46 | <menu name="HelpMenu" action="HelpMenu"> | 14 | <menu name="HelpMenu" action="HelpMenu"> |
157 | 47 | <menuitem name="About" action="CommonAbout" /> | 15 | <menuitem name="About" action="CommonAbout" /> |
158 | 48 | </menu> | 16 | </menu> |
159 | 49 | 17 | ||
160 | === modified file 'ui/event.ui' | |||
161 | --- ui/event.ui 2014-05-27 06:18:57 +0000 | |||
162 | +++ ui/event.ui 2014-08-28 02:36:47 +0000 | |||
163 | @@ -1,10 +1,6 @@ | |||
164 | 1 | <ui> | 1 | <ui> |
165 | 2 | 2 | ||
166 | 3 | <menubar name="MenuBar"> | 3 | <menubar name="MenuBar"> |
167 | 4 | <menu name="PhotosMenu" action="PhotosMenu"> | ||
168 | 5 | <separator /> | ||
169 | 6 | <menuitem name="MakePrimary" action="MakePrimary" /> | ||
170 | 7 | </menu> | ||
171 | 8 | </menubar> | 4 | </menubar> |
172 | 9 | 5 | ||
173 | 10 | <popup name="CollectionContextMenu"> | 6 | <popup name="CollectionContextMenu"> |
174 | @@ -17,4 +13,4 @@ | |||
175 | 17 | <menuitem name="EditEventComment" action="EditEventComment" /> | 13 | <menuitem name="EditEventComment" action="EditEventComment" /> |
176 | 18 | </popup> | 14 | </popup> |
177 | 19 | 15 | ||
179 | 20 | </ui> | 16 | </ui> |
180 | 21 | \ No newline at end of file | 17 | \ No newline at end of file |
181 | 22 | 18 | ||
182 | === modified file 'ui/events_directory.ui' | |||
183 | --- ui/events_directory.ui 2014-08-21 02:53:28 +0000 | |||
184 | +++ ui/events_directory.ui 2014-08-28 02:36:47 +0000 | |||
185 | @@ -4,19 +4,12 @@ | |||
186 | 4 | <menu name="FileMenu" action="FileMenu"> | 4 | <menu name="FileMenu" action="FileMenu"> |
187 | 5 | <menuitem name="Import" action="CommonFileImport" /> | 5 | <menuitem name="Import" action="CommonFileImport" /> |
188 | 6 | <menuitem name="ImportExternal" action="ExternalLibraryImport" /> | 6 | <menuitem name="ImportExternal" action="ExternalLibraryImport" /> |
189 | 7 | <separator /> | ||
190 | 8 | <menuitem name="EmptyTrash" action="CommonEmptyTrash" /> | ||
191 | 9 | <separator /> | ||
192 | 10 | <menuitem name="Quit" action="CommonQuit" /> | ||
193 | 11 | </menu> | 7 | </menu> |
194 | 12 | 8 | ||
195 | 13 | <menu name="EditMenu" action="EditMenu"> | 9 | <menu name="EditMenu" action="EditMenu"> |
196 | 14 | <menuitem name="Undo" action="CommonUndo" /> | 10 | <menuitem name="Undo" action="CommonUndo" /> |
197 | 15 | <menuitem name="Redo" action="CommonRedo" /> | 11 | <menuitem name="Redo" action="CommonRedo" /> |
198 | 16 | <separator /> | 12 | <separator /> |
199 | 17 | <menuitem name="Find" action="CommonFind" /> | ||
200 | 18 | <menuitem name="NewSearch" action="CommonNewSearch" /> | ||
201 | 19 | <separator /> | ||
202 | 20 | <menuitem name="Preferences" action="CommonPreferences" /> | 13 | <menuitem name="Preferences" action="CommonPreferences" /> |
203 | 21 | </menu> | 14 | </menu> |
204 | 22 | 15 | ||
205 | @@ -47,4 +40,4 @@ | |||
206 | 47 | <menuitem name="SelectAll" action="CommonSelectAll" /> | 40 | <menuitem name="SelectAll" action="CommonSelectAll" /> |
207 | 48 | </popup> | 41 | </popup> |
208 | 49 | 42 | ||
210 | 50 | </ui> | 43 | </ui> |
211 | 51 | \ No newline at end of file | 44 | \ No newline at end of file |
212 | 52 | 45 | ||
213 | === modified file 'ui/import.ui' | |||
214 | --- ui/import.ui 2014-08-21 02:53:28 +0000 | |||
215 | +++ ui/import.ui 2014-08-28 02:36:47 +0000 | |||
216 | @@ -6,18 +6,12 @@ | |||
217 | 6 | <menuitem name="ImportSelected" action="ImportSelected" /> | 6 | <menuitem name="ImportSelected" action="ImportSelected" /> |
218 | 7 | <menuitem name="ImportAll" action="ImportAll" /> | 7 | <menuitem name="ImportAll" action="ImportAll" /> |
219 | 8 | <menuitem name="ImportExternal" action="ExternalLibraryImport" /> | 8 | <menuitem name="ImportExternal" action="ExternalLibraryImport" /> |
220 | 9 | <separator /> | ||
221 | 10 | <menuitem name="EmptyTrash" action="CommonEmptyTrash" /> | ||
222 | 11 | <separator /> | ||
223 | 12 | <menuitem name="Quit" action="CommonQuit" /> | ||
224 | 13 | </menu> | 9 | </menu> |
225 | 14 | 10 | ||
226 | 15 | <menu name="EditMenu" action="EditMenu"> | 11 | <menu name="EditMenu" action="EditMenu"> |
227 | 16 | <menuitem name="Undo" action="CommonUndo" /> | 12 | <menuitem name="Undo" action="CommonUndo" /> |
228 | 17 | <menuitem name="Redo" action="CommonRedo" /> | 13 | <menuitem name="Redo" action="CommonRedo" /> |
229 | 18 | <separator /> | 14 | <separator /> |
230 | 19 | <menuitem name="Find" action="CommonFind" /> | ||
231 | 20 | <separator /> | ||
232 | 21 | <menuitem name="Preferences" action="CommonPreferences" /> | 15 | <menuitem name="Preferences" action="CommonPreferences" /> |
233 | 22 | </menu> | 16 | </menu> |
234 | 23 | 17 | ||
235 | 24 | 18 | ||
236 | === modified file 'ui/import_queue.ui' | |||
237 | --- ui/import_queue.ui 2014-08-05 14:06:52 +0000 | |||
238 | +++ ui/import_queue.ui 2014-08-28 02:36:47 +0000 | |||
239 | @@ -7,9 +7,6 @@ | |||
240 | 7 | <menuitem name="Import" action="CommonFileImport" /> | 7 | <menuitem name="Import" action="CommonFileImport" /> |
241 | 8 | <menuitem name="ImportExternal" action="ExternalLibraryImport" /> | 8 | <menuitem name="ImportExternal" action="ExternalLibraryImport" /> |
242 | 9 | <separator /> | 9 | <separator /> |
243 | 10 | <menuitem name="EmptyTrash" action="CommonEmptyTrash" /> | ||
244 | 11 | <separator /> | ||
245 | 12 | <menuitem name="Quit" action="CommonQuit" /> | ||
246 | 13 | </menu> | 10 | </menu> |
247 | 14 | 11 | ||
248 | 15 | <menu name="ViewMenu" action="ViewMenu"> | 12 | <menu name="ViewMenu" action="ViewMenu"> |
249 | @@ -24,4 +21,4 @@ | |||
250 | 24 | </menu> | 21 | </menu> |
251 | 25 | </menubar> | 22 | </menubar> |
252 | 26 | 23 | ||
254 | 27 | </ui> | 24 | </ui> |
255 | 28 | \ No newline at end of file | 25 | \ No newline at end of file |
256 | 29 | 26 | ||
257 | === modified file 'ui/media.ui' | |||
258 | --- ui/media.ui 2014-08-25 06:04:50 +0000 | |||
259 | +++ ui/media.ui 2014-08-28 02:36:47 +0000 | |||
260 | @@ -4,26 +4,15 @@ | |||
261 | 4 | <menu name="FileMenu" action="FileMenu"> | 4 | <menu name="FileMenu" action="FileMenu"> |
262 | 5 | <menuitem name="Import" action="CommonFileImport" /> | 5 | <menuitem name="Import" action="CommonFileImport" /> |
263 | 6 | <menuitem name="ImportExternal" action="ExternalLibraryImport" /> | 6 | <menuitem name="ImportExternal" action="ExternalLibraryImport" /> |
264 | 7 | <separator /> | ||
265 | 8 | <placeholder name="FileExtrasPlaceholder" /> | ||
266 | 9 | </menu> | 7 | </menu> |
267 | 10 | 8 | ||
268 | 11 | <menu name="EditMenu" action="EditMenu"> | 9 | <menu name="EditMenu" action="EditMenu"> |
269 | 12 | <menuitem name="Undo" action="CommonUndo" /> | 10 | <menuitem name="Undo" action="CommonUndo" /> |
270 | 13 | <menuitem name="Redo" action="CommonRedo" /> | 11 | <menuitem name="Redo" action="CommonRedo" /> |
271 | 14 | <separator /> | 12 | <separator /> |
272 | 15 | <placeholder name="EditExtrasPlaceholder" /> | ||
273 | 16 | <menuitem name="RemoveFromLibrary" action="RemoveFromLibrary" /> | ||
274 | 17 | <separator /> | ||
275 | 18 | <menuitem name="Preferences" action="CommonPreferences" /> | 13 | <menuitem name="Preferences" action="CommonPreferences" /> |
276 | 19 | </menu> | 14 | </menu> |
277 | 20 | 15 | ||
278 | 21 | <menu name="PhotosMenu" action="PhotosMenu"> | ||
279 | 22 | <placeholder name="PhotosExtrasEditsPlaceholder" /> | ||
280 | 23 | <separator /> | ||
281 | 24 | <placeholder name="PhotosExtrasExternalsPlaceholder" /> | ||
282 | 25 | </menu> | ||
283 | 26 | |||
284 | 27 | <placeholder name="FacesMenuPlaceholder" /> | 16 | <placeholder name="FacesMenuPlaceholder" /> |
285 | 28 | 17 | ||
286 | 29 | <menu name="HelpMenu" action="HelpMenu"> | 18 | <menu name="HelpMenu" action="HelpMenu"> |
287 | 30 | 19 | ||
288 | === modified file 'ui/offline.ui' | |||
289 | --- ui/offline.ui 2014-08-21 02:53:28 +0000 | |||
290 | +++ ui/offline.ui 2014-08-28 02:36:47 +0000 | |||
291 | @@ -14,11 +14,6 @@ | |||
292 | 14 | <menuitem name="Undo" action="CommonUndo" /> | 14 | <menuitem name="Undo" action="CommonUndo" /> |
293 | 15 | <menuitem name="Redo" action="CommonRedo" /> | 15 | <menuitem name="Redo" action="CommonRedo" /> |
294 | 16 | <separator /> | 16 | <separator /> |
295 | 17 | <menuitem name="Find" action="CommonFind" /> | ||
296 | 18 | <menuitem name="NewSearch" action="CommonNewSearch" /> | ||
297 | 19 | <separator /> | ||
298 | 20 | <menuitem name="RemoveFromLibrary" action="RemoveFromLibrary" /> | ||
299 | 21 | <separator /> | ||
300 | 22 | <menuitem name="Preferences" action="CommonPreferences" /> | 17 | <menuitem name="Preferences" action="CommonPreferences" /> |
301 | 23 | </menu> | 18 | </menu> |
302 | 24 | 19 | ||
303 | 25 | 20 | ||
304 | === modified file 'ui/photo.ui' | |||
305 | --- ui/photo.ui 2014-08-25 06:04:50 +0000 | |||
306 | +++ ui/photo.ui 2014-08-28 02:36:47 +0000 | |||
307 | @@ -10,8 +10,6 @@ | |||
308 | 10 | <menuitem name="Undo" action="CommonUndo" /> | 10 | <menuitem name="Undo" action="CommonUndo" /> |
309 | 11 | <menuitem name="Redo" action="CommonRedo" /> | 11 | <menuitem name="Redo" action="CommonRedo" /> |
310 | 12 | <separator /> | 12 | <separator /> |
311 | 13 | <menuitem name="RemoveFromLibrary" action="RemoveFromLibrary" /> | ||
312 | 14 | <separator /> | ||
313 | 15 | <menuitem name="Preferences" action="CommonPreferences" /> | 13 | <menuitem name="Preferences" action="CommonPreferences" /> |
314 | 16 | </menu> | 14 | </menu> |
315 | 17 | 15 | ||
316 | 18 | 16 | ||
317 | === modified file 'ui/photo_context.ui' | |||
318 | --- ui/photo_context.ui 2014-08-25 06:04:50 +0000 | |||
319 | +++ ui/photo_context.ui 2014-08-28 02:36:47 +0000 | |||
320 | @@ -37,6 +37,7 @@ | |||
321 | 37 | <menuitem name="ContextExport" action="Export" /> | 37 | <menuitem name="ContextExport" action="Export" /> |
322 | 38 | <placeholder name="ContractorPlaceholder" /> | 38 | <placeholder name="ContractorPlaceholder" /> |
323 | 39 | <separator /> | 39 | <separator /> |
324 | 40 | <menuitem name="RemoveFromLibrary" action="RemoveFromLibrary" /> | ||
325 | 40 | <menuitem name="ContextMoveToTrash" action="MoveToTrash" /> | 41 | <menuitem name="ContextMoveToTrash" action="MoveToTrash" /> |
326 | 41 | </popup> | 42 | </popup> |
327 | 42 | 43 | ||
328 | 43 | 44 | ||
329 | === modified file 'ui/tags.ui' | |||
330 | --- ui/tags.ui 2011-08-03 02:30:42 +0000 | |||
331 | +++ ui/tags.ui 2014-08-28 02:36:47 +0000 | |||
332 | @@ -1,13 +1,4 @@ | |||
333 | 1 | <ui> | 1 | <ui> |
334 | 2 | |||
335 | 3 | <menubar name="MenuBar"> | ||
336 | 4 | <menu name="TagsMenu" action="TagsMenu"> | ||
337 | 5 | <menuitem name="RemoveTagFromPhotos" action="RemoveTagFromPhotos" /> | ||
338 | 6 | <separator /> | ||
339 | 7 | <menuitem name="RenameTag" action="RenameTag" /> | ||
340 | 8 | <menuitem name="DeleteTag" action="DeleteTag" /> | ||
341 | 9 | </menu> | ||
342 | 10 | </menubar> | ||
343 | 11 | 2 | ||
344 | 12 | <popup name="CollectionContextMenu"> | 3 | <popup name="CollectionContextMenu"> |
345 | 13 | <placeholder name="ContextTagsPlaceholder"> | 4 | <placeholder name="ContextTagsPlaceholder"> |
346 | @@ -22,4 +13,4 @@ | |||
347 | 22 | <menuitem name="ContextRenameTag" action="DeleteTagSidebar" /> | 13 | <menuitem name="ContextRenameTag" action="DeleteTagSidebar" /> |
348 | 23 | </popup> | 14 | </popup> |
349 | 24 | 15 | ||
351 | 25 | </ui> | 16 | </ui> |
352 | 26 | \ No newline at end of file | 17 | \ No newline at end of file |
353 | 27 | 18 | ||
354 | === modified file 'ui/trash.ui' | |||
355 | --- ui/trash.ui 2014-08-21 02:53:28 +0000 | |||
356 | +++ ui/trash.ui 2014-08-28 02:36:47 +0000 | |||
357 | @@ -4,23 +4,12 @@ | |||
358 | 4 | <menu name="FileMenu" action="FileMenu"> | 4 | <menu name="FileMenu" action="FileMenu"> |
359 | 5 | <menuitem name="Import" action="CommonFileImport" /> | 5 | <menuitem name="Import" action="CommonFileImport" /> |
360 | 6 | <menuitem name="ImportExternal" action="ExternalLibraryImport" /> | 6 | <menuitem name="ImportExternal" action="ExternalLibraryImport" /> |
361 | 7 | <separator /> | ||
362 | 8 | <menuitem name="JumpToFile" action="CommonJumpToFile" /> | ||
363 | 9 | <menuitem name="EmptyTrash" action="CommonEmptyTrash" /> | ||
364 | 10 | <separator /> | ||
365 | 11 | <menuitem name="Quit" action="CommonQuit" /> | ||
366 | 12 | </menu> | 7 | </menu> |
367 | 13 | 8 | ||
368 | 14 | <menu name="EditMenu" action="EditMenu"> | 9 | <menu name="EditMenu" action="EditMenu"> |
369 | 15 | <menuitem name="Undo" action="CommonUndo" /> | 10 | <menuitem name="Undo" action="CommonUndo" /> |
370 | 16 | <menuitem name="Redo" action="CommonRedo" /> | 11 | <menuitem name="Redo" action="CommonRedo" /> |
371 | 17 | <separator /> | 12 | <separator /> |
372 | 18 | <menuitem name="Find" action="CommonFind" /> | ||
373 | 19 | <menuitem name="NewSearch" action="CommonNewSearch" /> | ||
374 | 20 | <separator /> | ||
375 | 21 | <menuitem name="Delete" action="Delete" /> | ||
376 | 22 | <menuitem name="Restore" action="Restore" /> | ||
377 | 23 | <separator /> | ||
378 | 24 | <menuitem name="Preferences" action="CommonPreferences" /> | 13 | <menuitem name="Preferences" action="CommonPreferences" /> |
379 | 25 | </menu> | 14 | </menu> |
380 | 26 | 15 | ||
381 | @@ -41,7 +30,7 @@ | |||
382 | 41 | <popup name="TrashPageMenu"> | 30 | <popup name="TrashPageMenu"> |
383 | 42 | <menuitem name="ContextEmptyTrash" action="CommonEmptyTrash" /> | 31 | <menuitem name="ContextEmptyTrash" action="CommonEmptyTrash" /> |
384 | 43 | </popup> | 32 | </popup> |
386 | 44 | 33 | ||
387 | 45 | <toolbar name="TrashToolbar"> | 34 | <toolbar name="TrashToolbar"> |
388 | 46 | <toolitem name="ToolDelete" action="Delete" /> | 35 | <toolitem name="ToolDelete" action="Delete" /> |
389 | 47 | <toolitem name="ToolRestore" action="Restore" /> | 36 | <toolitem name="ToolRestore" action="Restore" /> |
390 | @@ -74,4 +63,4 @@ | |||
391 | 74 | <menuitem name="SelectAll" action="CommonSelectAll" /> | 63 | <menuitem name="SelectAll" action="CommonSelectAll" /> |
392 | 75 | </popup> | 64 | </popup> |
393 | 76 | 65 | ||
395 | 77 | </ui> | 66 | </ui> |
396 | 78 | \ No newline at end of file | 67 | \ No newline at end of file |
Hm I'm experiencing a crash here that I'm not experiencing with the last daily build. Right-clicking on trash causes the crash:
src/Page. vala:1344: checkerboard_ page_real_ get_page_ sidebar_ menu: assertion failed: (menu != null)