Bug 107544

Summary: show context when doing PageUp PageDown
Product: [Applications] okular Reporter: Mikolaj Machowski <mikmach>
Component: generalAssignee: Okular developers <okular-devel>
Status: RESOLVED FIXED    
Severity: wishlist CC: aacid, bluedzins, ruanbeihong
Priority: VLO    
Version First Reported In: 0.8   
Target Milestone: ---   
Platform: Compiled Sources   
OS: Linux   
Latest Commit: Version Fixed In:
Sentry Crash Report:

Description Mikolaj Machowski 2005-06-16 18:28:31 UTC 
Version:            (using KDE Devel)
Installed from:    Compiled sources
OS:                Linux

At the moment kpdf is excuting PgUp/PgDn up to the pixel making very hard continous reading. Leave some context, 10-20 pixels, if possible to compute one line height.
Comment 1 Miroslav Ľos 2005-10-04 16:29:09 UTC 
I`d like it to be an option, so users could determine how many lines, pixels or percents of the viewport the context would be.
Comment 2 Mikolaj Machowski 2005-10-04 20:32:49 UTC 
> I`d like it to be an option, so users could determine how many
> lines, pixels or percents of the viewport the context would be.


Should be turned on by default with some sane value.
Comment 3 Mikolaj Machowski 2006-09-08 15:43:29 UTC 
Konqueror is doing it automatically, no need for additional option IMO.
Comment 4 Pino Toscano 2007-07-19 19:40:39 UTC 
*** Bug 127282 has been marked as a duplicate of this bug. ***
Comment 5 Mikolaj Machowski 2009-02-15 14:01:30 UTC 
Still valid for okular 4.2
Comment 6 James Ruan 2013-06-09 10:57:36 UTC 
The problem is still annoying in verion 0.14.
Comment 7 James Ruan 2013-06-10 04:55:49 UTC 
After looking into the code, I find Okular::Settings::scrollOverlap(), which is 0% by default. The programmer has considered the problem but didn't set the default value right (any value above 0 would be more appropriate I think).

Just for quick location:
void PageView::updatePageStep() {
    const QSize vs = viewport()->size();
    horizontalScrollBar()->setPageStep( vs.width() );
    verticalScrollBar()->setPageStep( vs.height() * (100 - Okular::Settings::scrollOverlap()) / 100 );
}
Comment 8 Albert Astals Cid 2013-06-10 18:01:01 UTC 
The programmer has set the default right value. You may have a different opinion and that's why you have an option to change it.