[Sugar-devel] [PATCH] Fix for PDF scrolling with Page Up/Page Down.
manuel quiñones
manuel.por.aca at gmail.com
Thu Jun 30 19:31:16 EDT 2011
El día 30 de junio de 2011 05:40, Simon Schampijer
<simon at schampijer.de> escribió:
> On 06/30/2011 06:32 AM, Manuel Quiñones wrote:
>>
>> We were passing wrong second parameter to the scroll method of
>> evince.View .
>> ---
>> evinceadapter.py | 4 ++--
>> 1 files changed, 2 insertions(+), 2 deletions(-)
>>
>> diff --git a/evinceadapter.py b/evinceadapter.py
>> index 3d3350c..3ddcfd4 100644
>> --- a/evinceadapter.py
>> +++ b/evinceadapter.py
>> @@ -228,9 +228,9 @@ class EvinceViewer():
>> _logger.error('scroll: %s', scrolltype)
>>
>> if scrolltype == gtk.SCROLL_PAGE_BACKWARD:
>> - self._view.scroll(gtk.SCROLL_PAGE_BACKWARD, -1)
>> + self._view.scroll(gtk.SCROLL_PAGE_BACKWARD, horizontal)
>> elif scrolltype == gtk.SCROLL_PAGE_FORWARD:
>> - self._view.scroll(gtk.SCROLL_PAGE_FORWARD, 1)
>> + self._view.scroll(gtk.SCROLL_PAGE_FORWARD, horizontal)
>> elif scrolltype == gtk.SCROLL_STEP_BACKWARD:
>> self._scroll_step(False)
>> elif scrolltype == gtk.SCROLL_STEP_FORWARD:
>
> Thanks for the patch. Looks good to me.
>
> Reviewed-by: Simon Schampijer <simon at laptop.org>
>
>
> Some small formal comments. If the patch fixes a bug, it is good to note
> that in the patch title (i.e. OLPC #10950, SL #123). And it is good to add
> yourself as the signer, like:
>
> Signed-off-by: Manuel Quiñones <manuq at laptop.org>
Thanks for pointing it Simon, I'll do next time.
--
.. manuq ..
More information about the Sugar-devel
mailing list