After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 170458 - Evince should provide a way to jump at the specified page on the command line
Evince should provide a way to jump at the specified page on the command line
Status: RESOLVED FIXED
Product: evince
Classification: Core
Component: general
0.1.x
Other All
: High enhancement
: ---
Assigned To: Evince Maintainers
Evince Maintainers
Depends on: 300641
Blocks:
 
 
Reported: 2005-03-15 14:47 UTC by Raphael Slinckx
Modified: 2005-05-20 05:48 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Patch to add a --page command-line option. (1.41 KB, patch)
2005-04-23 18:25 UTC, James Bowes
none Details | Review
Second attempt. (3.36 KB, patch)
2005-04-23 19:09 UTC, James Bowes
needs-work Details | Review
Patch (1.39 KB, patch)
2005-05-18 20:35 UTC, Nickolay V. Shmyrev
accepted-commit_now Details | Review

Description Raphael Slinckx 2005-03-15 14:47:23 UTC
It would be really nice if evince could jump to a specified page at opening,
through the command line for example

typing
evince --page 32 document.pdf

would open page 32 of the document, if there are at least 32 pages of course..

It would also allow, for example beagle to open a document in the exact place
where a word was found.

Going further, maybe also passing a find string on the command line could be
useful, to highlight the word ahen the document opens..
Comment 1 Raphael Slinckx 2005-03-15 14:57:58 UTC
As someone suggested on IRC, if evince could detect any change of the curent
file, and reload it at the same page and position when the file changes it would
be very useful. (via inotify or anything else)

This can be done in combination of the command line option...
Comment 2 Bryan W Clark 2005-03-16 06:55:20 UTC
Hi, thanks for the request.  I don't see much of a problem with doing this,
however it will probably be low in the priority queue without working patches. 
Also this is very similar to one part of bug 165242 

See the discussion on reload in bug 165119
Comment 3 James Bowes 2005-04-23 18:25:17 UTC
Created attachment 45592 [details] [review]
Patch to add a --page command-line option.
Comment 4 James Bowes 2005-04-23 19:09:32 UTC
Created attachment 45594 [details] [review]
Second attempt.

You can now provide either page numbers or page labels (i.e. iii, Index) on the
command-line.
Comment 5 Jonathan Blandford 2005-04-27 22:54:49 UTC
I don't think we want to keep the initial page on reload.  I think that would be
a little bit weird.  Other than that, patch looks good.
Comment 6 Nickolay V. Shmyrev 2005-05-12 14:22:03 UTC
Patch from bug 300641 has this work included and also solves the problems with
scrolling to page in continuous view
Comment 7 Nickolay V. Shmyrev 2005-05-18 08:36:38 UTC
Patch commited, thanks
Comment 8 Raphael Slinckx 2005-05-18 10:44:57 UTC
Thanks for that patch !
Comment 9 James Bowes 2005-05-18 14:00:13 UTC
The version of this patch included with the patch from bug 300641 does not work
with current CVS. Reopening the bug.
Comment 10 Nickolay V. Shmyrev 2005-05-18 20:35:48 UTC
Created attachment 46620 [details] [review]
Patch

Sorry for being so fast to close this bug. This patch should fix the problem
Comment 11 Nickolay V. Shmyrev 2005-05-18 20:37:10 UTC
Marco, it would be nice to have this patch in before the release
Comment 12 Marco Pesenti Gritti 2005-05-18 21:43:34 UTC
It would be nice if jrb could have a look in this one... I havent seen the new
view code at all yet.

Also, until the infinite loop is fixed I think we will have to delay the release :/
Comment 13 Marco Pesenti Gritti 2005-05-19 23:26:27 UTC
Comment on attachment 46620 [details] [review]
Patch

Looks sane to me even if I dont fully understand the pending_scroll logic.
Comment 14 Jonathan Blandford 2005-05-20 05:48:05 UTC
Committed, thanks.