Merge lp:~codygarver/audience/fix-1380819 into lp:~audience-members/audience/trunk

Proposed by Cody Garver
Status: Merged
Approved by: Cody Garver
Approved revision: 424
Merged at revision: 431
Proposed branch: lp:~codygarver/audience/fix-1380819
Merge into: lp:~audience-members/audience/trunk
Diff against target: 56 lines (+13/-11)
3 files modified
src/Audience.vala (+12/-1)
src/CMakeLists.txt (+1/-2)
src/desktop-launcher.vala (+0/-8)
To merge this branch: bzr merge lp:~codygarver/audience/fix-1380819
Reviewer Review Type Date Requested Status
Tom Beckmann codestyle & code Approve
Review via email: mp+239169@code.launchpad.net

Commit message

Add some and move .desktop translatable strings from desktop-launcher.vala to Audience.vala to prepare for bug #1380819

To post a comment you must log in.
lp:~codygarver/audience/fix-1380819 updated
423. By Cody Garver

QUICKLIST_ABOUT is not actually a constant

424. By Cody Garver

8 spaces because it's inside class

Revision history for this message
Tom Beckmann (tombeckmann) wrote :

Code looks good.

review: Approve (codestyle & code)

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'src/Audience.vala'
2--- src/Audience.vala 2014-10-20 16:38:01 +0000
3+++ src/Audience.vala 2014-10-23 09:32:31 +0000
4@@ -31,8 +31,19 @@
5
6 public class App : Granite.Application {
7
8+ /**
9+ * Translatable launcher (.desktop) strings to be added to template (.pot) file.
10+ * These strings should reflect any changes in these launcher keys in .desktop file
11+ */
12+ public const string VIDEOS = N_("Videos");
13+ public const string PROGRAM_NAME = "Audience";
14+ public const string KEYWORDS = N_("Audience;Video;Player;Movies;");
15+ public const string COMMENT = N_("Watch videos and movies");
16+ public const string GENERIC_NAME = N_("Video Player");
17+ public string QUICKLIST_ABOUT = N_("About ") + PROGRAM_NAME;
18+
19 construct {
20- program_name = "Audience";
21+ program_name = PROGRAM_NAME;
22 exec_name = "audience";
23
24 build_data_dir = Constants.DATADIR;
25
26=== modified file 'src/CMakeLists.txt'
27--- src/CMakeLists.txt 2014-08-13 03:12:48 +0000
28+++ src/CMakeLists.txt 2014-10-23 09:32:31 +0000
29@@ -62,7 +62,6 @@
30 Widgets/Playlist.vala
31 Widgets/PlaylistPopover.vala
32 Widgets/VideoPlayer.vala
33- desktop-launcher.vala
34 PACKAGES
35 ${VALA_DEPS}
36 OPTIONS
37@@ -75,4 +74,4 @@
38
39 add_executable (${EXEC_NAME} ${VALA_C})
40 target_link_libraries (${EXEC_NAME} ${DEPS_LIBRARIES})
41-install (TARGETS ${EXEC_NAME} RUNTIME DESTINATION bin)
42\ No newline at end of file
43+install (TARGETS ${EXEC_NAME} RUNTIME DESTINATION bin)
44
45=== removed file 'src/desktop-launcher.vala'
46--- src/desktop-launcher.vala 2014-10-14 00:41:22 +0000
47+++ src/desktop-launcher.vala 1970-01-01 00:00:00 +0000
48@@ -1,8 +0,0 @@
49-namespace DesktopLauncher {
50- /* Translatable launcher (.desktop) strings to be added to */
51- /* template (.pot) file. These strings MUST reflect any */
52- /* changes in these launcher keys in .desktop file. */
53- public const string COMMENT = N_("Watch videos and movies");
54- public const string GENERIC_NAME = N_("Video Player");
55- public const string ABOUT = N_("About Audience");
56-}

Subscribers

People subscribed via source and target branches

to all changes: