Merge lp:~clissold345/ubuntu-docs/fix-for-1569532 into lp:~ubuntu-core-doc/ubuntu-docs/trunk2017-07-10

Proposed by Chris Perry
Status: Merged
Merged at revision: 589
Proposed branch: lp:~clissold345/ubuntu-docs/fix-for-1569532
Merge into: lp:~ubuntu-core-doc/ubuntu-docs/trunk2017-07-10
Diff against target: 101 lines (+75/-0)
3 files modified
ubuntu-help/C/addremove-creator.page (+73/-0)
ubuntu-help/C/addremove.page (+1/-0)
ubuntu-help/Makefile.am (+1/-0)
To merge this branch: bzr merge lp:~clissold345/ubuntu-docs/fix-for-1569532
Reviewer Review Type Date Requested Status
Gunnar Hjalmarsson Approve
Doug Smythies Needs Fixing
Review via email: mp+313960@code.launchpad.net

Description of the change

This all seemed pretty straightforward.

The terminology situation is a bit messy. Eg I noticed that the gui uses "start-up" at one point but I've used "startup" throughout. I've used "USB flash drive" not "USB stick", etc. I've used "CD image" not "ISO file", etc.

I like to give indications of time, where it's possible to do so. Here it is and I have: "ten minutes or less".

I haven't reminded the user to (safely) eject the USB flash drive. Perhaps I should?

To post a comment you must log in.
Revision history for this message
Doug Smythies (dsmythies) wrote :

Hi Chris,

Thanks for your continuing good work on the help docs.
This MP seems to include the previous MP, which is O.K., I guess.

Since you are adding a new file the revision version should be set accordingly.
You will also need to edit Makefile.am and add the file.

I'll actually build the docs and look at the content later.

review: Needs Fixing
587. By Chris Perry

Describe the Startup Disk Creator; LP: #1569532

Revision history for this message
Chris Perry (clissold345) wrote :

Hi Doug, thanks for your reply. Yes I've set revision to 17.04 (since that's the release the page will be released in). Is that correct? I've also set the date to today's date. Is that correct? Yes I've updated Makefile.am. My two merge proposals are independent so presumably there's no reason why (once approved) one can't be merged in without the other?

Revision history for this message
Gunnar Hjalmarsson (gunnarhj) wrote :

Besides Doug's comment, which I agree on, I don't think that this page fits well in the "Install & remove software" section. I would suggest that it's renamed and placed in the "Disks & storage" section instead, which deals with things affecting whole partitions.

As regards the contents, the term "CD image" sounds a bit dated considering that most images don't fit on a CD nowadays. I suggest "DVD image" or just "image". Also, I think that the term "ISO file" should be mentioned at least once.

These are the changes I propose:

gunnar@gunnar-ubuntu-current:/media/gunnar/ubuntu-dev/home/gunnar/UbuntuDev/bzr/ubuntu-docs/trunk/ubuntu-help/C$ diff -u addremove-creator.page disk-creator.page
--- addremove-creator.page 2017-01-02 20:41:02.965027000 +0100
+++ disk-creator.page 2017-01-02 21:54:22.961657670 +0100
@@ -1,6 +1,6 @@
 <page xmlns="http://projectmallard.org/1.0/"
       type="topic" style="task"
- id="addremove-create">
+ id="disk-creator">

   <info>
     <credit type="author">
@@ -12,8 +12,7 @@
     </desc>

     <include href="legal.xml" xmlns="http://www.w3.org/2001/XInclude"/>
- <link type="guide" xref="addremove" group="#first"/>
- <link type="seealso" xref="addremove-install-synaptic"/>
+ <link type="guide" xref="disk"/>
     <revision version="16.04" date="2016-03-13" status="review"/>
   </info>

@@ -25,9 +24,9 @@
   The capacity of the USB flash drive must be at least 2 GB.
   Any data on the USB flash drive will be lost when you create the startup disk.</p>

- <p>It's assumed that you have downloaded an Ubuntu CD image.</p>
+ <p>It's assumed that you have downloaded an Ubuntu image (ISO file).</p>

- <p>To create a startup disk from an Ubuntu CD image:</p>
+ <p>To create a startup disk from an Ubuntu image:</p>

   <steps>
     <item>
@@ -37,8 +36,8 @@
     </item>
     <item>
       <p>
- If the CD image is in your <gui>Download</gui> folder and there is just one CD image, <app>Startup Disk Creator</app> will have selected it,
- otherwise select the correct CD image.
+ If the image is in your <gui>Download</gui> folder and there is just one image, <app>Startup Disk Creator</app> will have selected it,
+ otherwise select the correct image.
       </p>
     </item>
     <item>

review: Needs Fixing
Revision history for this message
Chris Perry (clissold345) wrote :

Hi Gunnar, thanks for your reply. Your points about "CD image" and "ISO file" are fine. I haven't read all of the diff output but I'm assuming there's nothing in it that you didn't mention in your first two paragraphs.

I'm fine to have a link to the new page on disk.html but I'd argue that the link I (in effect) added on addremove.html makes sense too - because the ultimate task for many of the people who want to create a startup disk is to install software (that is, to install Ubuntu). (I'll probably revise my new page slightly to make it clear that one can install Ubuntu using a startup disk.) What do you think?

Presumably it's not a problem to link to a page from two "menu" pages? If necessary we just have two copies of the page?

Revision history for this message
Gunnar Hjalmarsson (gunnarhj) wrote :

Even if you in the next step 'install software' in a sense, I still think it's a bit far-fetched. But ok, there is no obvious place in the current structure for this page, so let's link from both. But it shouldn't be the first item under "Install & remove software", so I think the group value should be "#last" instead of "#first".

<link type="guide" xref="disk"/>
<link type="guide" xref="addremove" group="#last"/>

I agree that installing Ubuntu should be mentioned somehow. Maybe by including a link to <https://www.ubuntu.com/download/desktop/install-ubuntu-desktop>.

588. By Chris Perry

Describe the Startup Disk Creator; LP: #1569532

Revision history for this message
Chris Perry (clissold345) wrote :

Hi Gunnar, thanks for taking the time to review this.

Wikipedia likes "ISO image" so I've used that term - but for the first two uses I've expanded it to "Ubuntu ISO image".

https://en.wikipedia.org/wiki/ISO_image

Initially I misunderstood one of your comments and thought I had to change addremove.page. I seem to have accidentally added a blank line to addremove.page - but it doesn't matter if that change goes in or not.

The new page (addremove-create.page) is now on both "menu" pages, etc.

I hope I've covered everything.

589. By Chris Perry

Describe the Startup Disk Creator; LP: #1569532

Revision history for this message
Chris Perry (clissold345) wrote :

One more change: I removed the "see also" link, which comes from the page I copied and is not needed here.

Revision history for this message
Gunnar Hjalmarsson (gunnarhj) wrote :

One tiny detail: Please change the page id from "addremove-create" to "addremove-creator" so it matches the file name.

Besides that it looks good, so assuming that you make that change before pushing I approve now.

Thanks!

review: Approve
Revision history for this message
Chris Perry (clissold345) wrote :

Thanks. I've made the change and pushed to trunk. bzr says "Pushed up to revision 590".

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== added file 'ubuntu-help/C/addremove-creator.page'
2--- ubuntu-help/C/addremove-creator.page 1970-01-01 00:00:00 +0000
3+++ ubuntu-help/C/addremove-creator.page 2017-01-03 22:30:38 +0000
4@@ -0,0 +1,73 @@
5+<page xmlns="http://projectmallard.org/1.0/"
6+ type="topic" style="task"
7+ id="addremove-create">
8+
9+ <info>
10+ <credit type="author">
11+ <name>Ubuntu Documentation Team</name>
12+ </credit>
13+
14+ <desc>
15+ Convert a USB flash drive into a volume from which you can startup and install Ubuntu.
16+ </desc>
17+
18+ <include href="legal.xml" xmlns="http://www.w3.org/2001/XInclude"/>
19+
20+ <link type="guide" xref="disk"/>
21+ <link type="guide" xref="addremove" group="#last"/>
22+
23+ <revision version="17.04" date="2017-01-02" status="review"/>
24+ </info>
25+
26+ <title>
27+ Create a startup disk
28+ </title>
29+
30+ <p>In this context a startup disk is a USB flash drive from which you can startup and <link
31+ href="https://www.ubuntu.com/download/desktop/install-ubuntu-desktop">install Ubuntu</link>.
32+ The capacity of the USB flash drive must be at least 2 GB.
33+ Any data on the USB flash drive will be lost when you create the startup disk.</p>
34+
35+ <p>It's assumed that you have downloaded an Ubuntu ISO image.</p>
36+
37+ <p>To create a startup disk from an Ubuntu ISO image:</p>
38+
39+ <steps>
40+ <item>
41+ <p>
42+ Open <app>Startup Disk Creator</app> from the <gui>Dash</gui>.
43+ </p>
44+ </item>
45+ <item>
46+ <p>
47+ If the ISO image is in your <gui>Download</gui> folder and there is just one ISO image, <app>Startup Disk Creator</app> will have selected it,
48+ otherwise select the correct ISO image.
49+ </p>
50+ </item>
51+ <item>
52+ <p>
53+ If your computer has just one USB flash drive, <app>Startup Disk Creator</app> will have selected it, otherwise select the correct drive.
54+ </p>
55+ </item>
56+ <item>
57+ <p>
58+ Click <gui>Make Startup Disk</gui>.
59+ </p>
60+ </item>
61+ <item>
62+ <p>
63+ Click <gui>Yes</gui> to confirm. The process should take ten minutes or less.
64+ </p>
65+ </item>
66+
67+ <item>
68+ <p>
69+ When the "Installation Complete" window opens click <gui>Quit</gui>.
70+ </p>
71+ </item>
72+
73+ </steps>
74+
75+
76+
77+</page>
78
79=== modified file 'ubuntu-help/C/addremove.page'
80--- ubuntu-help/C/addremove.page 2016-11-08 21:06:37 +0000
81+++ ubuntu-help/C/addremove.page 2017-01-03 22:30:38 +0000
82@@ -5,6 +5,7 @@
83 <revision version="14.04" date="2013-12-26" status="review"/>
84
85 <link type="guide" xref="index" group="addremove"/>
86+
87 <title role="trail" type="link">Add/remove software</title>
88
89 <credit type="author">
90
91=== modified file 'ubuntu-help/Makefile.am'
92--- ubuntu-help/Makefile.am 2016-09-14 21:14:53 +0000
93+++ ubuntu-help/Makefile.am 2017-01-03 22:30:38 +0000
94@@ -60,6 +60,7 @@
95 accounts-remove.page \
96 accounts-which-application.page \
97 accounts-whyadd.page \
98+ addremove-creator.page \
99 addremove-install-synaptic.page \
100 addremove-install.page \
101 addremove-ppa.page \

Subscribers

People subscribed via source and target branches