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 661598 - Full screen preview should have an Edit option
Full screen preview should have an Edit option
Status: RESOLVED FIXED
Product: gnome-documents
Classification: Core
Component: general
0.2.x
Other Linux
: Normal normal
: ---
Assigned To: GNOME documents maintainer(s)
GNOME documents maintainer(s)
: 661708 (view as bug list)
Depends on:
Blocks:
 
 
Reported: 2011-10-12 21:46 UTC by Cosimo Cecchi
Modified: 2012-07-19 21:33 UTC
See Also:
GNOME target: ---
GNOME version: ---



Description Cosimo Cecchi 2011-10-12 21:46:58 UTC
The full screen preview should have an Edit option in its action toolbar item.
That would show an OSD toolbar over the document, with options for rotating pages; possibly zoom too, even if we want to support that by pinch gestures probably.
Comment 1 Cosimo Cecchi 2011-10-13 20:26:48 UTC
*** Bug 661708 has been marked as a duplicate of this bug. ***
Comment 2 Matthias Clasen 2012-07-05 22:39:47 UTC
Regardless of fullscreen or not, the context menu in the preview is a bit too hard to discover. I've seen people complain that there is no way to start editing from the preview. Also, Print should be available in the same way.

Other problems of the context menu:

- One-item menus are awkward

- Weak contrast with the whitish preview
Comment 3 Cosimo Cecchi 2012-07-19 21:33:17 UTC
The context menu is now gone in git master, and a toolbar menu has been implemented with the suggested actions:

- Open
- Print
- Zoom in/out
- Rotate left/right

The only thing that could possibly be improved is using an explicit "Edit" verb for the opening action if we know we're going to be using Libreoffice, but since we show the name of the application anwyay, this is probably not a big deal.
I'm going to close this as FIXED.