Review of attachment 221302 [details] [review]:

This page needs to be added to the help/Makefile.am and retake the screenshot using the Aidwaita theme, but it would actually be better not to have the screenshot as they adds lots of extra work for translators.

In general the content is fine, but here are some point on which it can be improved:

::: help/C/history.page
@@ +1,3 @@
+<page xmlns="http://projectmallard.org/1.0/"
+      type="guide"
+      id="history">

<page xmlns="http://projectmallard.org/1.0/"
      xmlns:its="http://www.w3.org/2005/11/its"
      type="topic" style="task"
      id="history">

@@ +5,3 @@
+    <info>
+        <link type="guide" xref="index#ui"/>
+    </info>

<info>
    <link type="guide" xref="index#ui />
    <revision pkgversion="3.8" version="0.1" date="2013-04-18" status="review"/>

    <credit type="author copyright">
      <name>Gopal Krishnan</name>
      <email its:translate="no">ingeniumed@outlook.com</email>
      <years>2012, 2013</years>
    </credit>

  </info>

@@ +14,3 @@
+    The History View is located above the entry box and is designed to show your previous calculations in the form of a scrollable list 
+    that can be clicked and scrolled through using a mouse or even navigated and selected using the arrow keys on a keyboard. Once an 
+    entry has been selected, it will immediately show up in the entry box for you to manipulate and use. Finally, there isn't an upper 

"is not", it is better not to use contractions.

@@ +16,3 @@
+    entry has been selected, it will immediately show up in the entry box for you to manipulate and use. Finally, there isn't an upper 
+    limit on the number of calculations that can show up in the list for you to use.
+    </p>

It would be great if you could reflow this paragraph to 79 characters wide.

This patch depends of the merging of the patch at Bug #327590, as it describes a feature implemented there.

Created attachment 344439 [details] [review]
Fixed patch for history page

This has been inactive for quite some time, so I took this up.

I have fixed the suggestions that were given in the previous review, removed the image and added the entry to help/Makefile.am.

Since most of the content was written by the previous author, I was not sure of what to do in the <credit> tag, so I left it as suggested.

Review of attachment 344439 [details] [review]:

Thanks for taking this, it has been neglected indeed, your contributions are welcome. Please take into account a couple of minor nitpicks, and I'll be ready to push your next version with those fixed.

Review of attachment 344439 [details] [review]:

And now with the minor change requests.

::: help/C/history.page
@@ +23,3 @@
+    using the arrow keys on a keyboard. Once an entry has been selected, it
+    will immediately show up in the entry box for you to manipulate and use.
+    Finally, there isn't an upper limit on the number of calculations that can

As the previous review stated, contractions should not be used, so this should be something like "there is no upper limit ..."

::: help/Makefile.am
@@ +19,3 @@
              financial.page \
              functions.page \
+	     history.page \

Please use spaces for indenting, instead of mixed spaces and tabs.

Created attachment 345556 [details] [review]
Fixed patch

- Add myself as author
- Fix grammar
- Never mix spaces and tabs!

Hi Robert,

I have updated the patch.

Thanks.

This problem has been fixed in the unstable development version. The fix will be available in the next major software release. You may need to upgrade your Linux distribution to obtain that newer version.