Bug 107544 - show context when doing PageUp PageDown
Summary: show context when doing PageUp PageDown
Status: RESOLVED FIXED
Alias: None
Product: okular
Classification: Applications
Component: general (show other bugs)
Version: 0.8
Platform: Compiled Sources Linux
: VLO wishlist
Target Milestone: ---
Assignee: Okular developers
URL:
Keywords:
: 127282 (view as bug list)
Depends on:
Blocks:
 
Reported: 2005-06-16 18:28 UTC by Mikolaj Machowski
Modified: 2013-06-10 18:01 UTC (History)
3 users (show)

See Also:
Latest Commit:
Version Fixed In:
Sentry Crash Report:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
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.