GNOME Bugzilla – Bug 737010
Corrected some grammar mistakes in 'connect.page','create.page','system-requirements.page', and simplifies/clarified 'create.page'. No related bug.
Last modified: 2016-03-31 13:22:07 UTC
The documentation is more clear particularly in the installation sources list on the create.page. I have not contributed to the GNOME project before, so please let me know if I'm doing anything wrong.
Created attachment 286677 [details] [review] Corrected some grammar mistakes in 'connect.page','create.page','system-requirements.page', and simplifies/clarified 'create.page'. No related bug.
Thanks for your page! In general very welcome. :) One small question: A local virtual machine + can be created from:</p> - <item><p>inserting installation media in your CD/DVD drive or USB - port.</p></item> + <item><p>A CD/DVD/USB drive.</p></item> This change sounds like I can create a virtual machine from a hardware drive, but isn't it rather the media inserted in that drive?
Review of attachment 286677 [details] [review]: Thanks for the patch! ::: help/C/create.page @@ +47,3 @@ </item> + <item><p>After making your selection, and providing any necessary "or" was correct here. You can click Continue after selecting an ISO or entering a URL. If you do either one, no further information is entered until after clicking Continue.
@André Klapper: You are right, it should be more specific. I'll upload a new patch shortly with a fix! @Michael Hill: You are also right, I've changed that back to how you guys had it before, which will also be included in the new patch I'll upload in a bit. Thank you guys for the comments! There'll be a new patch shortly.
Created attachment 286701 [details] [review] Fixes issues with original patch from bug 737010
Comment on attachment 286701 [details] [review] Fixes issues with original patch from bug 737010 Looks good, Alexander. Are you able to commit this?
A small thing: I think it would be nice to have something more meaningful in the commit message so we don't have to look into the commit or into this bug to get what's this about. Welcome from me too! And thanks for the patch.
I am not able to commit (or push, if that's what you mean) these changes, as I do not have those privileges, as far as I know. If I'm missing something, just let me know. I'll make sure to be more meaningful in my commit message in the future.
Created attachment 286739 [details] [review] help: fix, clarify connect, create, sys-reqs pages
Attachment 286739 [details] pushed as 7e7a20b - help: fix, clarify connect, create, sys-reqs pages