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 648115 - Evince doesn't exits presentation mode at the end of presentation
Evince doesn't exits presentation mode at the end of presentation
Status: RESOLVED OBSOLETE
Product: evince
Classification: Core
Component: general
2.32.x
Other Linux
: Normal minor
: ---
Assigned To: Evince Maintainers
Evince Maintainers
Depends on:
Blocks:
 
 
Reported: 2011-04-18 13:55 UTC by Gaurav Maheshwari
Modified: 2018-05-22 14:12 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
patch fixing the bug. compiled and tested against the Git repository (2.04 KB, patch)
2011-07-15 12:32 UTC, Alexandre Negrao
needs-work Details | Review
rebase (2.04 KB, patch)
2011-07-15 19:12 UTC, Alexandre Negrao
none Details | Review
new patch (1.04 KB, patch)
2011-07-18 13:49 UTC, Alexandre Negrao
none Details | Review

Description Gaurav Maheshwari 2011-04-18 13:55:15 UTC
When I open a document in presentation mode, at the end of the presentation, I get a black screen showing "End of presentation, Click to Exit". Pressing down arrow button should exit this presentation (as it happens in OpenOffice/MS Office), but it doesn't exit. I have to click to end the presentation.
Comment 1 Alexandre Negrao 2011-07-15 12:32:09 UTC
Created attachment 192025 [details] [review]
patch fixing the bug. compiled and tested against the Git repository

I would like to develop additional features,
how can I submit my commits to you through Git?
What is the preferred format for the evince team?

Best Regards!
Comment 2 Hib Eris 2011-07-15 18:12:28 UTC
Review of attachment 192025 [details] [review]:

Please squash the two patches using 'git rebase -i master'.
Comment 3 Alexandre Negrao 2011-07-15 19:12:36 UTC
Created attachment 192055 [details] [review]
rebase

Done!

Thanks!
Comment 4 Hib Eris 2011-07-18 12:26:18 UTC
I think I was not clear enough: Please squash the second patch onto the first using 'git rebase -i master' and than choosing to squash, thus merging patch 1 and patch 2 into one proper patch.
Comment 5 Hib Eris 2011-07-18 12:29:18 UTC
(please ignore the hyperlinks in my previous comment, those were added by bugzilla and are not pointing to anything relevant.)
Comment 6 Alexandre Negrao 2011-07-18 13:49:05 UTC
Created attachment 192184 [details] [review]
new patch

Thanks for the help!
Im still new at this. Somehow rebase didnt work.
Made a new patch without need of corrections.
Comment 7 Hib Eris 2011-07-26 15:56:41 UTC
I am wondering whether we really want to make this change.

Pro:
- Compatible with OpenOffice/MSOffice.
- Exiting presentation mode is more obvious for any user.

Contra:
- I actually like the fact that I can keep the PageDown button pressed until Evince reaches the end of a presentation to go through the presentation on high speed, without exiting presentation mode. Exiting presentation mode is something a user should do consciously and deliberately. 

An alternative could be that we change the text from "End of presentation. Click to exit." to "End of presentation. Press Esc or click to exit."

Anyone else have an opinion on this?
Comment 8 Christian Reis (not reading bugmail) 2011-09-02 20:48:24 UTC
While I think you make a good point about the action of exiting the presentation be deliberate, I tend to feel that the compatibility of OOO/MSOffice might bear more weight in this situation. Either way, I suspect it's not going to be that big a deal and probably worth trying the patch as-is since it's already posted.
Comment 9 GNOME Infrastructure Team 2018-05-22 14:12:21 UTC
-- GitLab Migration Automatic Message --

This bug has been migrated to GNOME's GitLab instance and has been closed from further activity.

You can subscribe and participate further through the new bug through this link to our GitLab instance: https://gitlab.gnome.org/GNOME/evince/issues/213.