[gnome-klotski/mallard-help] Review Commit

[Rashi Aswani <aswanirashi src gnome org>]

commit 1f704aaf52859b8862cabf827cb4fec538dbec80
Author: Sindhu S <sindhus gnome org>
Date:   Mon Aug 25 12:33:12 2014 -0600

    Review Commit

 help/C/index.page              |    3 +++
 help/C/keyboard-shortcuts.page |    4 ++--
 help/C/moves.page              |    5 +++--
 help/C/rules.page              |    5 +++--
 help/C/start.page              |    2 ++
 5 files changed, 13 insertions(+), 6 deletions(-)
---
diff --git a/help/C/index.page b/help/C/index.page
index 6fde127..109b3a5 100644
--- a/help/C/index.page
+++ b/help/C/index.page
@@ -3,6 +3,7 @@
   id="index"
   type="guide"
   style="2column">
+<!--Can't they not be in the same line? -->
 
   <info>
     <title type="link" role="trail"><media type="image" its:translate="no"
@@ -24,6 +25,8 @@
   Klotski logo</media> Klotski</title>
   <p>Slide blocks to solve the puzzle. To know more about the game, see
   <link xref="introduction">Game introduction</link>.</p>
+  <!-- See gEdit's help, see how they introduced their introduction pages?
+Am not happy with the second line here.-->
 
   <section id="play" style="2column">
     <title>Playing Klotski</title>
diff --git a/help/C/keyboard-shortcuts.page b/help/C/keyboard-shortcuts.page
index 87abcef..9839f73 100644
--- a/help/C/keyboard-shortcuts.page
+++ b/help/C/keyboard-shortcuts.page
@@ -14,12 +14,12 @@
     <include href="legal.xml" xmlns="http://www.w3.org/2001/XInclude"; />
 
     <desc>Play the game through keyboard.</desc>
-
+<!-- not through, its "using" keyboard -->
   </info>
 
   <title>Keyboard shortcuts</title>
   <p>To access the game through keyboard, use the following shortcuts:</p>
-
+<!-- look at how jim's written intro about keyboard access in gedit's index.pageand revise this -->
     <table frame="all" rules="cols" shade="rows">
       <tr>
         <td><p>New game</p></td>
diff --git a/help/C/moves.page b/help/C/moves.page
index 2563869..a261834 100644
--- a/help/C/moves.page
+++ b/help/C/moves.page
@@ -16,11 +16,11 @@
     <include href="legal.xml" xmlns="http://www.w3.org/2001/XInclude"; />
 
     <desc>Check the number of moves you made.</desc>
-
+<!-- Say a line about "A move is blah blah and you can move them towards the goal piece" you know what am 
hinting at?-->
   </info>
 
   <title>Moves</title>
-
+<!-- starting the setence with "The" is not correct, revise -->
   <p>The consecutive moves of a block are considered to be a single move. For
   example, moving a block up one square then right two squares is considered one
   move.</p>
@@ -31,5 +31,6 @@
       goal piece, which is the patterned block, is always permitted to move to
       it. However, no other blocks are permitted to move off the board.</p>
   </section>
+  <!-- What is the meaning "permitted to move to it", what is it? -->
  
 </page>
diff --git a/help/C/rules.page b/help/C/rules.page
index b4071dc..b90fdd3 100644
--- a/help/C/rules.page
+++ b/help/C/rules.page
@@ -20,10 +20,11 @@
   </info>
 
   <title>Game rules</title>
-
+<!--Might want to a spell check, what's patterened? :) -->
   <p>The aim of <app>Klotski</app> is to move the patterened block to a
    specific location. Some simple rules you must follow while moving:</p>
-  
+  <!-- Not happy with the second sentence, revise it please. why do we two sections here? one is just fine. 
Merge them. -->
+
   <section id="move-block">
     <title>Move a block</title>
     <p>Click and drag the desired block to a location of your choice on the
diff --git a/help/C/start.page b/help/C/start.page
index 2f8ac29..2d7dfcc 100644
--- a/help/C/start.page
+++ b/help/C/start.page
@@ -1,6 +1,8 @@
 <page xmlns="http://projectmallard.org/1.0/";
       type="topic"
       id="start">
+
+<!-- this page should show up near introduction not in the playing klotski section -->
   <info>
     <link type="guide" xref="index#play"/>
     <link type="seealso" xref="rules"/>