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 735446 - Evince headerbar buttons have gone missing
Evince headerbar buttons have gone missing
Status: RESOLVED FIXED
Product: evince
Classification: Core
Component: general
3.13.x
Other Linux
: Normal normal
: ---
Assigned To: Evince Maintainers
Evince Maintainers
: 735527 (view as bug list)
Depends on:
Blocks:
 
 
Reported: 2014-08-26 11:42 UTC by Kalev Lember
Modified: 2014-08-28 14:00 UTC
See Also:
GNOME target: 3.14
GNOME version: ---


Attachments
shell: Ensure toolbar widgets are shown when constructing the toolbar (938 bytes, patch)
2014-08-26 13:40 UTC, José Aliste
rejected Details | Review
shell: set toolbar to Normal Mode in set_document (1.47 KB, patch)
2014-08-26 21:22 UTC, José Aliste
committed Details | Review

Description Kalev Lember 2014-08-26 11:42:18 UTC
After updating to evince 3.13.90, all the buttons in the headerbar have gone missing. There's no gear (hamburger) menu, no find button, no zoom options.

Looks like this only happens when opening a file directly ('evince myfile.pdf' or through nautilus); when I go through the Recent Documents page all the buttons appear as expected.

https://kalev.fedorapeople.org/evince-no-buttons.png
Comment 1 Peter Robinson 2014-08-26 11:48:04 UTC
also if you hit F-11 and go to full screen mode the buttons appear
Comment 2 José Aliste 2014-08-26 13:40:39 UTC
Created attachment 284509 [details] [review]
shell: Ensure toolbar widgets are shown when constructing the toolbar

In some cases we were not calling gtk_widget_show on some of the widgets
in the toolbar.
Comment 3 Carlos Garcia Campos 2014-08-26 15:23:19 UTC
Review of attachment 284509 [details] [review]:

::: shell/ev-toolbar.c
@@ +278,3 @@
 
+	/* Ensure toolbar widgets are shown */
+	ev_toolbar_set_mode (ev_toolbar, ev_toolbar->priv->toolbar_mode);

I don't think this is the best approach. I think it should be the window, the one calling ev_toolbar_set_mode(). The actual problem is the early return added to ev_window_destroy_recent_view(). I think we should call ev_toolbar_set_mode() in ev_window_set_document() but also move ev_window_title_set_type() and ev_window_update_actions_sensitivity() from ev_window_destroy_recent_view() to ev_window_set_document().
Comment 4 José Aliste 2014-08-26 15:42:27 UTC
Review of attachment 284509 [details] [review]:

::: shell/ev-toolbar.c
@@ +278,3 @@
 
+	/* Ensure toolbar widgets are shown */
+	ev_toolbar_set_mode (ev_toolbar, ev_toolbar->priv->toolbar_mode);

I Agree
Comment 5 José Aliste 2014-08-26 21:22:57 UTC
Created attachment 284552 [details] [review]
shell: set toolbar to Normal Mode in set_document

setting the toolbar to normal mode will correctly
show widgets inside the toolbar
Comment 6 Michael Catanzaro 2014-08-27 15:52:35 UTC
*** Bug 735527 has been marked as a duplicate of this bug. ***
Comment 7 Carlos Garcia Campos 2014-08-28 13:21:12 UTC
Comment on attachment 284552 [details] [review]
shell: set toolbar to Normal Mode in set_document

Thanks
Comment 8 José Aliste 2014-08-28 14:00:31 UTC
Review of attachment 284552 [details] [review]:

ok
Comment 9 José Aliste 2014-08-28 14:00:47 UTC
This problem has been fixed in the development version. The fix will be available in the next major software release. Thank you for your bug report.