[gnome-klotski/mallard-help] Review Commit
- From: Rashi Aswani <aswanirashi src gnome org>
- To: commits-list gnome org
- Cc:
- Subject: [gnome-klotski/mallard-help] Review Commit
- Date: Tue, 26 Aug 2014 08:10:19 +0000 (UTC)
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"/>
[
Date Prev][
Date Next] [
Thread Prev][
Thread Next]
[
Thread Index]
[
Date Index]
[
Author Index]