Merge lp:~tim-clicks/ubuntu-docs/minor into lp:ubuntu-docs/lucid
Status: | Rejected |
---|---|
Rejected by: | Connor Imes |
Proposed branch: | lp:~tim-clicks/ubuntu-docs/minor |
Merge into: | lp:ubuntu-docs/lucid |
Diff against target: |
60 lines (+19/-11) 1 file modified
newtoubuntu/C/newtoubuntu.xml (+19/-11) |
To merge this branch: | bzr merge lp:~tim-clicks/ubuntu-docs/minor |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Connor Imes (community) | Disapprove | ||
Review via email: mp+21764@code.launchpad.net |
Description of the change
Largely cosmetic, but overall, beneficial changes
Added the occasional comma. Discarded semi-colons in favour of new
sentences. Attempted to shorten sentences where beneficial.
Non-trivial changes:
location - action - justification
l 78 - deleted some text - Sounded a little too much like an advert.
l 79 - removed 'update' - Not accurate
l 87 - content changes - Provided some more specifics, used 'non-free'
Validation:
$ bash ./scripts/
--Validating ./newtoubuntu/
yelp seems to render well.
Unmerged revisions
- 483. By Tim McNamara
-
Edited introductory paragraphs
Actions & justifications:
"New to Ubuntu?"
Removed redundancy in "New to Ubuntu"?
Appeared twice: once in header, then in first sentence
Expanded the first sentance into listitems
Attempted to use more of the page, so the contents didn't
have such a visually dominating effect."Welcome to Ubuntu"
Spread content out to two sentences
There were two messages. One was about Ubuntu, the other
was about getting new software. This commit provides a
visual seperation, so that the messages are clearer. - 482. By Tim McNamara
-
Largely cosmetic changes
Added the occasional comma. Discarded semi-colons in favour of new
sentences. Attempted to shorten sentences where beneficial.Non-trivial changes:
location - action - justification
l 78 - deleted some text - Sounded a little too much like an advert.
l 79 - removed 'update' - Not accurate
l 87 - content changes - Provided some more specifics, used 'non-free'
over 'proprietary' because it's more accurate
Rejected in favor of https:/ /code.launchpad .net/~tim- clicks/ ubuntu- docs/minor2/ +merge/ 21767