Merge lp:~jwcampbell/deja-dup/deja-dup-doc-updates into lp:deja-dup/28

Proposed by Jim Campbell
Status: Merged
Approved by: Michael Terry
Approved revision: 1459
Merge reported by: Michael Terry
Merged at revision: not available
Proposed branch: lp:~jwcampbell/deja-dup/deja-dup-doc-updates
Merge into: lp:deja-dup/28
Diff against target: 224 lines (+25/-18)
12 files modified
help/C/backup-auto.page (+1/-1)
help/C/backup-first.page (+1/-1)
help/C/contribute.page (+1/-1)
help/C/credits.page (+5/-5)
help/C/index.page (+8/-1)
help/C/license.page (+1/-1)
help/C/prefs.page (+3/-3)
help/C/restore-full.page (+1/-1)
help/C/restore-lost.page (+1/-1)
help/C/restore-revert.page (+1/-1)
help/C/restore-worst-case.page (+1/-1)
help/C/support.page (+1/-1)
To merge this branch: bzr merge lp:~jwcampbell/deja-dup/deja-dup-doc-updates
Reviewer Review Type Date Requested Status
Michael Terry Approve
Review via email: mp+170892@code.launchpad.net

Description of the change

Some trivial fixes to the Deja Dup documentation. Most of the fixes are related to style, but I did fix a validation error on credits.page, as well as a few minor typos.

To post a comment you must log in.
Revision history for this message
Michael Terry (mterry) wrote :

Thanks so much for the fixes!

A) I don't like the change from "Backup Help" to icon / "Deja Dup Backup Tool". The icon isn't well-aligned and I'd prefer to not use the deja dup name quite so prominently.

B) I don't notice the effect of the style="tip" or style="task" changes. Is it a subtle change or something that my theme might not be doing?

C) For Alternatively->Alternately, I think Alternatively actually makes more sense. See http://www.wordnik.com/words/alternately vs http://www.wordnik.com/words/alternatively

D) Why do you prefer "regularly" to be after the verb rather than in front?

E) Good catch on "services costs" and the missing id= attributes.

review: Needs Fixing
1459. By Jim Campbell

docs: increased size of deja-dup icon. changed title back to 'backup help'. changed 'alternately' back to 'alternatively'

Revision history for this message
Jim Campbell (jwcampbell) wrote :

Hi Michael,

Here are my notes based on your prior comments:

A) I don't like the change from "Backup Help" to icon / "Deja Dup Backup Tool". The icon isn't well-aligned and I'd prefer to not use the deja dup name quite so prominently.

-- I've changed this back.

B) I don't notice the effect of the style="tip" or style="task" changes. Is it a subtle change or something that my theme might not be doing?

-- It's a bit hard to explain in writing, but when the 'style="tip"' or other style attribute is missing, it looks as if an image in the "favicon" area of yelp tried to load, but couldn't. It isn't a big deal, but I just think it looks a bit better to use the style attribute.

C) For Alternatively->Alternately, I think Alternatively actually makes more sense. See http://www.wordnik.com/words/alternately vs http://www.wordnik.com/words/alternatively

-- I've reverted this change.

D) Why do you prefer "regularly" to be after the verb rather than in front?

-- Thanks for asking. Per this page [0], including the adverb at the end of the sentence makes it stronger. Given that we're encouraging people to back up more frequently, I thought it gave it a little more oomph to put "reguarly" at the end. (Both approaches are technically fine, though).

E) Good catch on "services costs" and the missing id= attributes.

-- : )

==========
One other thing I noticed was that the docs don't break the line at 79 characters, and this makes diffs hard to read in Launchpad (and in other diff-related tools). With GNOME docs, we prefer to break the lines at 79 chars where possible just because it makes diffs easier to read. I wanted to hold off on making that change until I got an OK from you, though.

If it's ok with you, I'll make the change to each of the doc pages (breaking the lines at 79 chars where I can) w/o making any other textual changes. Let me know what you think.

[0] http://www.perfect-english-grammar.com/adverbs-of-frequency.html

Revision history for this message
Michael Terry (mterry) wrote :

100 + <title type="link">Backup Help</title>
101 + <title type="text">Backup Help</title>

This is duplicated, I assume accidentally.

111 + <media type="image" mime="image/png" src="figures/deja-dup-icon.png">
112 + </media>

This doesn't seem well-centered in my tests. Plus, none of the other GNOME apps seem to use an icon. I'm inclined to drop this change.

Revision history for this message
Jim Campbell (jwcampbell) wrote :

> 100 + <title type="link">Backup Help</title>
> 101 + <title type="text">Backup Help</title>
>
> This is duplicated, I assume accidentally.

The repetition is intentional. See here for further information: http://projectmallard.org/1.0/mal_info_title

This approach is used in gnome-user-docs, gedit, file-roller, the draft seahorse help, and other gnome help.

>
> 111 + <media type="image" mime="image/png" src="figures/deja-dup-
> icon.png">
> 112 + </media>
>
> This doesn't seem well-centered in my tests. Plus, none of the other GNOME
> apps seem to use an icon. I'm inclined to drop this change.

I'm using the same approach as used in gedit, gnome-help, ubuntu help, seahorse (aka passwords and keys), gnome-terminal, and the other gnome applications will be getting their appropriate icon in this manner in the near future. I see what you mean about it not lining up exactly right - not sure what I can do about it, as the alignment is handled by the yelp stylesheets.

Thanks for being so thorough in your review. Let me know if you have other fixes you'd like me to make.

Revision history for this message
Jim Campbell (jwcampbell) wrote :

If you'd prefer, I can resubmit without the icon. You had just noted that you were inclined to drop this change, so I am not sure if you meant the entire changeset I submitted, or just the icon.

Here is more information on use of multiple title links: http://projectmallard.org/pipermail/mallard-list/2013-July/000167.html

Cheers

Revision history for this message
Michael Terry (mterry) wrote :

Sorry this took me so long. :(

Approved, thanks for your work and putting up with my changes and delays!

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'help/C/backup-auto.page'
2--- help/C/backup-auto.page 2011-08-28 00:50:59 +0000
3+++ help/C/backup-auto.page 2013-07-17 21:23:25 +0000
4@@ -1,6 +1,6 @@
5 <?xml version="1.0" encoding="utf-8"?>
6 <page xmlns="http://projectmallard.org/1.0/"
7- type="topic"
8+ type="topic" style="task"
9 id="backup-auto">
10 <info>
11 <desc>Give yourself peace of mind</desc>
12
13=== modified file 'help/C/backup-first.page'
14--- help/C/backup-first.page 2011-05-06 21:59:19 +0000
15+++ help/C/backup-first.page 2013-07-17 21:23:25 +0000
16@@ -1,6 +1,6 @@
17 <?xml version="1.0" encoding="utf-8"?>
18 <page xmlns="http://projectmallard.org/1.0/"
19- type="topic"
20+ type="topic" style="task"
21 id="backup-first">
22 <info>
23 <desc>Start now</desc>
24
25=== modified file 'help/C/contribute.page'
26--- help/C/contribute.page 2011-06-13 18:55:47 +0000
27+++ help/C/contribute.page 2013-07-17 21:23:25 +0000
28@@ -2,7 +2,7 @@
29 <page xmlns="http://projectmallard.org/1.0/"
30 xmlns:its="http://www.w3.org/2005/11/its"
31 its:version="1.0"
32- type="topic"
33+ type="topic" style="task"
34 id="contribute">
35 <info>
36 <desc>Help make <app>Déjà Dup</app> better</desc>
37
38=== modified file 'help/C/credits.page'
39--- help/C/credits.page 2011-06-28 13:36:58 +0000
40+++ help/C/credits.page 2013-07-17 21:23:25 +0000
41@@ -2,7 +2,7 @@
42 <page xmlns="http://projectmallard.org/1.0/"
43 xmlns:its="http://www.w3.org/2005/11/its"
44 its:version="1.0"
45- type="topic"
46+ type="topic" style="tip"
47 id="credits">
48 <info>
49 <desc>Meet the team</desc>
50@@ -10,7 +10,7 @@
51 </info>
52 <title>Credits</title>
53
54-<section>
55+<section id="development-credits">
56 <title>Development</title>
57 <list>
58 <item its:translate="no"><p><link href="mailto:temposs@gmail.com">Andrew Fister</link></p></item>
59@@ -21,7 +21,7 @@
60 </list>
61 </section>
62
63-<section>
64+<section id="documentation-credits">
65 <title>Documentation</title>
66 <list>
67 <item its:translate="no"><p><link href="mailto:mike@mterry.name">Michael Terry</link></p></item>
68@@ -29,7 +29,7 @@
69 </list>
70 </section>
71
72-<section>
73+<section id="art-credits">
74 <title>Art</title>
75 <list>
76 <item its:translate="no"><p><link href="mailto:nisses.mail@home.se">Andreas Nilsson</link></p></item>
77@@ -39,7 +39,7 @@
78 </list>
79 </section>
80
81-<section>
82+<section id="translation-credits">
83 <title>Translation</title>
84 <code>translator-credits</code>
85 </section>
86
87=== added directory 'help/C/figures'
88=== added file 'help/C/figures/deja-dup-icon.png'
89Binary files help/C/figures/deja-dup-icon.png 1970-01-01 00:00:00 +0000 and help/C/figures/deja-dup-icon.png 2013-07-17 21:23:25 +0000 differ
90=== modified file 'help/C/index.page'
91--- help/C/index.page 2011-04-21 09:30:07 +0000
92+++ help/C/index.page 2013-07-17 21:23:25 +0000
93@@ -3,8 +3,11 @@
94 xmlns:its="http://www.w3.org/2005/11/its"
95 its:version="1.0"
96 type="guide"
97+ style="2column task"
98 id="index">
99 <info>
100+ <title type="link">Backup Help</title>
101+ <title type="text">Backup Help</title>
102 <credit type="author maintainer" its:translate="no">
103 <name>Michael Terry</name>
104 <email>mike@mterry.name</email>
105@@ -13,7 +16,11 @@
106 <license href="http://creativecommons.org/licenses/by-sa/3.0/"><p>Creative Commons Attribution-Share Alike 3.0 Unported License</p></license>
107 </info>
108
109-<title>Backup Help</title>
110+ <title>
111+ <media type="image" mime="image/png" src="figures/deja-dup-icon.png">
112+ </media>
113+ Backup Help
114+ </title>
115
116 <section id="backup" style="2column">
117 <title>Backing Up</title>
118
119=== modified file 'help/C/license.page'
120--- help/C/license.page 2011-04-30 02:37:05 +0000
121+++ help/C/license.page 2013-07-17 21:23:25 +0000
122@@ -1,6 +1,6 @@
123 <?xml version="1.0" encoding="utf-8"?>
124 <page xmlns="http://projectmallard.org/1.0/"
125- type="topic"
126+ type="topic" style="tip"
127 id="license">
128 <info>
129 <desc>Know your rights</desc>
130
131=== modified file 'help/C/prefs.page'
132--- help/C/prefs.page 2013-02-05 15:55:29 +0000
133+++ help/C/prefs.page 2013-07-17 21:23:25 +0000
134@@ -1,7 +1,7 @@
135 <page xmlns="http://projectmallard.org/1.0/"
136 xmlns:its="http://www.w3.org/2005/11/its"
137 its:version="1.0"
138- type="topic"
139+ type="topic" style="task"
140 id="prefs">
141 <info>
142 <desc>Adjust your backup settings</desc>
143@@ -11,7 +11,7 @@
144 <terms>
145 <item>
146 <title><gui>Automatic backups</gui></title>
147- <p>Turn this option on to have <app>Déjà Dup</app> automatically back up for you. This is recommended so that you don’t forget to do it yourself. Backups are more useful the more recent they are, so it is important to regularly back up.</p>
148+ <p>Turn this option on to have <app>Déjà Dup</app> automatically back up for you. This is recommended so that you don’t forget to do it yourself. Backups are more useful the more recent they are, so it is important to back up regularly.</p>
149 </item>
150 <item>
151 <title><gui>Backup location</gui></title>
152@@ -20,7 +20,7 @@
153 <item>
154 <title>Cloud Services</title>
155 <p>The first few backup locations are cloud storage services provided by various companies. For a small fee, they hold your data for you. A cloud service like this is recommended because it is an easy way to keep your data offsite (meaning out of your home: in case of disaster or theft, your data will still be safe).</p>
156- <note style="warning"><p>These services costs money. Read their rates carefully before using them.</p></note>
157+ <note style="warning"><p>These services cost money. Read their rates carefully before using them.</p></note>
158 </item>
159 <item>
160 <title>Remote Servers</title>
161
162=== modified file 'help/C/restore-full.page'
163--- help/C/restore-full.page 2011-07-22 13:15:59 +0000
164+++ help/C/restore-full.page 2013-07-17 21:23:25 +0000
165@@ -1,6 +1,6 @@
166 <?xml version="1.0" encoding="utf-8"?>
167 <page xmlns="http://projectmallard.org/1.0/"
168- type="topic"
169+ type="topic" style="task"
170 id="restore-full">
171 <info>
172 <desc>Recover from total system failure</desc>
173
174=== modified file 'help/C/restore-lost.page'
175--- help/C/restore-lost.page 2011-07-22 13:15:59 +0000
176+++ help/C/restore-lost.page 2013-07-17 21:23:25 +0000
177@@ -2,7 +2,7 @@
178 <page xmlns="http://projectmallard.org/1.0/"
179 xmlns:its="http://www.w3.org/2005/11/its"
180 its:version="1.0"
181- type="topic"
182+ type="topic" style="task"
183 id="restore-lost">
184 <info>
185 <desc>Restore a file that is no longer present</desc>
186
187=== modified file 'help/C/restore-revert.page'
188--- help/C/restore-revert.page 2011-08-15 18:21:37 +0000
189+++ help/C/restore-revert.page 2013-07-17 21:23:25 +0000
190@@ -2,7 +2,7 @@
191 <page xmlns="http://projectmallard.org/1.0/"
192 xmlns:its="http://www.w3.org/2005/11/its"
193 its:version="1.0"
194- type="topic"
195+ type="topic" style="task"
196 id="restore-revert">
197 <info>
198 <desc>Restore a previous version of a file</desc>
199
200=== modified file 'help/C/restore-worst-case.page'
201--- help/C/restore-worst-case.page 2011-06-28 13:36:58 +0000
202+++ help/C/restore-worst-case.page 2013-07-17 21:23:25 +0000
203@@ -2,7 +2,7 @@
204 <page xmlns="http://projectmallard.org/1.0/"
205 xmlns:its="http://www.w3.org/2005/11/its"
206 its:version="1.0"
207- type="topic"
208+ type="topic" style="task"
209 id="restore-worst-case">
210 <info>
211 <desc>What to do if you can’t restore your files</desc>
212
213=== modified file 'help/C/support.page'
214--- help/C/support.page 2011-05-06 21:59:19 +0000
215+++ help/C/support.page 2013-07-17 21:23:25 +0000
216@@ -2,7 +2,7 @@
217 <page xmlns="http://projectmallard.org/1.0/"
218 xmlns:its="http://www.w3.org/2005/11/its"
219 its:version="1.0"
220- type="topic"
221+ type="topic" style="tip"
222 id="support">
223 <info>
224 <desc>Get more help</desc>

Subscribers

People subscribed via source and target branches