Thanks by the patch and the test.<div>Pushed</div><div><br></div><div>Gonzalo<br><br><div class="gmail_quote">On Fri, Jun 1, 2012 at 9:02 AM, Simon Schampijer <span dir="ltr"><<a href="mailto:simon@schampijer.de" target="_blank">simon@schampijer.de</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Thanks Manuel for the patch.<br>
<br>
The patch does work for me. Gonzalo knows better the Read code to do a proper review.<br>
<br>
Tested-by: Simon Schampijer <<a href="mailto:simon@laptop.org" target="_blank">simon@laptop.org</a>><div><div class="h5"><br>
<br>
<br>
On 05/31/2012 05:08 PM, Manuel Kaufmann wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Show the bookmark for the first page (if any) in the sidebar when the<br>
file is resumed from the Journal.<br>
<br>
Signed-off-by: Manuel Kaufmann<<a href="mailto:humitos@gmail.com" target="_blank">humitos@gmail.com</a>><br>
---<br>
  readactivity.py |    5 +++++<br>
  1 file changed, 5 insertions(+)<br>
<br>
diff --git a/readactivity.py b/readactivity.py<br>
index 5e23b86..7cbe065 100644<br>
--- a/readactivity.py<br>
+++ b/readactivity.py<br>
@@ -834,6 +834,11 @@ class ReadActivity(activity.<u></u>Activity):<br>
          self._view.set_current_page(<u></u>current_page)<br>
          self._update_nav_buttons(<u></u>current_page)<br>
<br>
+        # README: bookmark sidebar is not showing the bookmark in the<br>
+        # first page because this is updated just if the page number changes<br>
+        if current_page == 0:<br>
+            self._sidebar.update_for_page(<u></u>current_page)<br>
+<br>
          # We've got the document, so if we're a shared activity, offer it<br>
          try:<br>
              if self.get_shared():<br>
</blockquote>
<br></div></div>
______________________________<u></u>_________________<br>
Sugar-devel mailing list<br>
<a href="mailto:Sugar-devel@lists.sugarlabs.org" target="_blank">Sugar-devel@lists.sugarlabs.<u></u>org</a><br>
<a href="http://lists.sugarlabs.org/listinfo/sugar-devel" target="_blank">http://lists.sugarlabs.org/<u></u>listinfo/sugar-devel</a><br>
</blockquote></div><br></div>