[tali/wip/mallard-help-merge: 35/49] Review: fix all

[Ekaterina Gerasimova <egerasimov src gnome org>]

commit 58de24c15736e0bd396b38643f9f2372fdf3171a
Author: Sindhu S <sindhus live in>
Date:   Mon Jul 14 21:58:38 2014 -0400

    Review: fix all

 help/C/game-settings.page      |    2 ++
 help/C/keyboard-shortcuts.page |    1 +
 help/C/play-game.page          |    4 ++++
 help/C/players-setting.page    |    1 +
 help/C/scores.page             |    1 +
 help/C/settings.page           |    2 +-
 6 files changed, 10 insertions(+), 1 deletions(-)
---
diff --git a/help/C/game-settings.page b/help/C/game-settings.page
index 2a83484..c901b4a 100644
--- a/help/C/game-settings.page
+++ b/help/C/game-settings.page
@@ -28,8 +28,10 @@
       <gui style="menuitem">Preferences</gui></guiseq>.</p></item>
       <item><p>You can select the game version you want to play from the drop
       down menu next to <gui>Game Type</gui>.</p></item>
+      <!-- Not digging the language, avoid "You can blah blah", just start with "Select the desired version 
of the game from <gui>..." -->
       <item><p>To start playing the newly selected version of the game, you
       need to <link xref="play-game">Start a new game</link>.</p></item>
+      <!-- language, language! Just say "Now, you may <link> Start a new.. " -->
     </steps>
 </page>
 
diff --git a/help/C/keyboard-shortcuts.page b/help/C/keyboard-shortcuts.page
index 4f2464a..547c515 100644
--- a/help/C/keyboard-shortcuts.page
+++ b/help/C/keyboard-shortcuts.page
@@ -20,6 +20,7 @@
   <title>Keyboard shortcuts</title>
   <p>You can play <app>tali</app> using the keyboard keys as well.</p>
   <list>
+    <!--this needs to be in a table -->
     <item><p>You can choose the dice to roll using keys 1 through 5, the key 1
     corresponds to the first die and so on.</p></item>
     <item><p>Switch to the <gui>Roll!</gui> button using <key>Tab</key>, press
diff --git a/help/C/play-game.page b/help/C/play-game.page
index ea1794e..9a80da0 100644
--- a/help/C/play-game.page
+++ b/help/C/play-game.page
@@ -1,6 +1,7 @@
 <page xmlns="http://projectmallard.org/1.0/";
       type="topic"
       id="play-game">
+<!--The page says start a new game, your id is play-game, can we have a little consistency here please? 
change page id to start-game or something -->
 
   <info>
     <link type="guide" xref="index#play"/>
@@ -41,6 +42,7 @@
     <item><p>Click on the dice you wish to roll. You can select all the dice to
     roll in the second or third roll by clicking <gui>Roll all!</gui> on the
     bottom left.</p></item>
+    <!-- you dont have to indicate the areas in the screen!! remove "on the bottom left" -->
     <item><p>Click on <gui>Roll!</gui> to roll the selected dice.</p></item>
     <item><p>When you have a hand you like or have run out of chances to throw
     the dice, click on a row corresponding to what you think is your best
@@ -58,11 +60,13 @@
     Kind</gui> a second time, you can place it in the <gui>5 of a Kind</gui>
     row again for fifty points.You can keep doing this each time you get
     another <gui>5 of a  Kind</gui>.</p>
+    <!--to which rule is the exception?? dont split notes! -->
   </note>
 
   <note style="tip">
     <p>There is also a bonus of 35 points if you score a total of
     63 points or more in the top six rows.</p>
+    <!-- Avoid the word "also", rewrite this sentence -->
   </note>
   </section>
 </page>
diff --git a/help/C/players-setting.page b/help/C/players-setting.page
index 72e973a..8fdc0b5 100644
--- a/help/C/players-setting.page
+++ b/help/C/players-setting.page
@@ -1,6 +1,7 @@
 <page xmlns="http://projectmallard.org/1.0/";
       type="topic"
       id="players-setting">
+      <!-- can we not have a single page for settings? since all of it is in tali > preferences -->
 
   <info>
     <link type="guide" xref="index#advanced"/>
diff --git a/help/C/scores.page b/help/C/scores.page
index bc90e35..ff28b89 100644
--- a/help/C/scores.page
+++ b/help/C/scores.page
@@ -27,6 +27,7 @@
       <item><p> The dialog will display the <gui>Regular</gui> version by
       default, to view scores obtained in <gui>Colors</gui>, select it from the
       drop down menu at the top of the dialog.</p></item>
+      <!-- whats with you describing the UI! the gui tag is to markup the UI visually so dont say things 
like "dialog" or point out where in the UI it is like "bottom left" and stuff. rewrite this sentence -->
     </steps>
 
   <section id="score-slots">
diff --git a/help/C/settings.page b/help/C/settings.page
index 3b71f79..70a12cd 100644
--- a/help/C/settings.page
+++ b/help/C/settings.page
@@ -1,7 +1,7 @@
 <page xmlns="http://projectmallard.org/1.0/";
       type="topic"
       id="settings">
-
+<!-- there are two pages on players settings, fix this! -->
   <info>
     <link type="guide" xref="index#advanced"/>
     <revision version="3.12" date="2014-06-14" status="review"/>