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 724549 - Navigation buttons (history) disabled after one click on back/forward
Navigation buttons (history) disabled after one click on back/forward
Status: RESOLVED FIXED
Product: evince
Classification: Core
Component: general
git master
Other Linux
: Normal normal
: ---
Assigned To: Evince Maintainers
Evince Maintainers
Depends on:
Blocks:
 
 
Reported: 2014-02-17 14:11 UTC by Anuj Khare
Modified: 2014-02-23 17:11 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Removes the lines that cause the history navigation buttons to become insensitive (465 bytes, patch)
2014-02-18 14:15 UTC, Anuj Khare
none Details | Review
Don't update history buttons sensitivity on page change (995 bytes, patch)
2014-02-18 17:59 UTC, Anuj Khare
none Details | Review
Make histroy buttons insensitive when searching (1.54 KB, patch)
2014-02-21 23:25 UTC, Anuj Khare
none Details | Review
Make histroy buttons insensitive when searching (1.62 KB, patch)
2014-02-23 07:42 UTC, Anuj Khare
committed Details | Review

Description Anuj Khare 2014-02-17 14:11:14 UTC
If I:

1. Navigate to some pages such that they are added to the history (|page_new - page_old| > 1).
2. Click on "Go to previous history item" button.

Result:
The history navigation buttons get disabled.

Expected result:
Navigation buttons remain sensitive to allow going forward from there or back in history further (if possible).

They are made sensitive again after a page change.

Similar behavior happens when I use the "Go to next history item" button.
Comment 1 Anuj Khare 2014-02-17 14:16:34 UTC
I think the issue arises because of the fix for this bug :

https://bugzilla.gnome.org/show_bug.cgi?id=704184
(commit 878979a833d03c53c61a2ad8ef6e9879632104cb)
Comment 2 Anuj Khare 2014-02-18 14:15:57 UTC
Created attachment 269553 [details] [review]
Removes the lines that cause the history navigation buttons to become insensitive

Removes the lines that cause the history navigation buttons to become insensitive because of the page-change signal being emitted while history is frozen every time we go back/forward in history.

However, this solution would require to reopen bug https://bugzilla.gnome.org/show_bug.cgi?id=704184.
Comment 3 Anuj Khare 2014-02-18 17:59:27 UTC
Created attachment 269588 [details] [review]
Don't update history buttons sensitivity on page change

Sorry for the previous attachment.
Comment 4 José Aliste 2014-02-21 12:30:24 UTC
Review of attachment 269588 [details] [review]:

We can't commit a patch to fix a bug that we know reopens another bug.
Comment 5 Anuj Khare 2014-02-21 23:25:41 UTC
Created attachment 269959 [details] [review]
Make histroy buttons insensitive when searching

Fixes this bug and while taking into account https://bugzilla.gnome.org/show_bug.cgi?id=704184.
Comment 6 Anuj Khare 2014-02-23 07:42:44 UTC
Created attachment 270041 [details] [review]
Make histroy buttons insensitive when searching

Updated the commit message.
Comment 7 Carlos Garcia Campos 2014-02-23 17:11:34 UTC
Comment on attachment 270041 [details] [review]
Make histroy buttons insensitive when searching

Thanks, pushed to git master