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 541424 - Make the sidebar look like EOG's
Make the sidebar look like EOG's
Status: RESOLVED FIXED
Product: evince
Classification: Core
Component: general
2.22.x
Other Linux
: Normal enhancement
: ---
Assigned To: Evince Maintainers
Evince Maintainers
Depends on: 646055 680953
Blocks:
 
 
Reported: 2008-07-03 18:29 UTC by Pedro Villavicencio
Modified: 2013-02-18 05:41 UTC
See Also:
GNOME target: ---
GNOME version: 2.21/2.22


Attachments
(preliminary) patch which makes the thumbnail selection in the sidebar look (almost) like EOG's (10.71 KB, patch)
2008-10-25 13:39 UTC, Christian Spurk
none Details | Review
Evince' sidebar look with my (preliminary) patch applied (44.44 KB, image/png)
2008-10-25 13:41 UTC, Christian Spurk
  Details

Description Pedro Villavicencio 2008-07-03 18:29:25 UTC
this bug has been filed here:

https://bugs.edge.launchpad.net/ubuntu/+source/evince/+bug/230700

"Evince and EOG are, in a way, similar apps: their purpose is to view specific kinds of documents.

Having their sidebars look the same would increase visual integration. I think Evince's sidebar should look like EOG's, mimicking the rounded borders that nautilus has too.
"

Mockup:

http://launchpadlibrarian.net/14539868/Pantallazo-1.png

Thanks,
Comment 1 Nickolay V. Shmyrev 2008-07-03 21:54:33 UTC
Thanks, nice idea
Comment 2 Christian Spurk 2008-10-25 13:39:57 UTC
Created attachment 121340 [details] [review]
(preliminary) patch which makes the thumbnail selection in the sidebar look (almost) like EOG's

This patch adds the rounded selection rectangle to the sidebar's thumbnails (see screenshot in the next attachment). The selection itself resembles exactly the way EOG's selections look like.

The only problem with this patch is that the page number of each thumbnail is not centered under the thumbnail anymore. For some reason I can't get the GtkIconView to do this with my custom cell renderer for the thumbnail. I'd gratefully embrace any pertinent hints on how to achieve this :-)

While this patch is an improvement to the current situation imho, it's a matter of taste whether it could be applied as it is or not. In any case, I'm also going to try some other approaches to getting this bug fixed with the page numbers centered under the thumbnails.
Comment 3 Christian Spurk 2008-10-25 13:41:43 UTC
Created attachment 121341 [details]
Evince' sidebar look with my (preliminary) patch applied
Comment 4 Christian Spurk 2008-10-25 13:43:15 UTC
(adding myself to CC list)
Comment 5 Carlos Garcia Campos 2009-10-26 10:55:15 UTC
GtkIconView already has a rounded selection rectangle, isn't it enough? or can we close this bug now?
Comment 6 Tobias Mueller 2010-03-22 14:32:16 UTC
Christian, can you comment on Carlos' question in comment #5?

Carlos, if I am not mistaken, then Evince does already do what this bug reports tries to fix, no? If so, could we close this bug as OBSOLETE?
Comment 7 Christian Spurk 2010-03-22 22:23:59 UTC
Oh, I must have overlooked Carlos’ question.

As for the GtkIconView I’m not sure whether there is actually any simple way to force a rounded selection rectangle if the current theme uses no rounded corners for selections. Probably that is also the reason why EOG had a custom cell renderer until at least the 2.26 series. Since 2.28 this custom cell renderer appears to be removed in EOG, though.

And this probably also makes this bug obsolete. Now the selection rectangle only depends on the currently used theme for both apps. Thereby either both Evince and EOG have rounded selection rectangles for their thumbnails or none of them has.
Comment 8 Robert Roth 2010-11-29 13:59:38 UTC
What is the status of this bug? If it's obsolete, it should be marked as such.
Comment 9 Carlos Garcia Campos 2011-06-19 10:47:24 UTC
Does eog still have its own cell renderer? maybe we could use it as a workaround for bug #646055, although a patch to fix GtkIconView in GTK 3.0 would be much better.
Comment 10 Yuri Myasoedov 2012-11-09 12:11:12 UTC
I've checked this issue in Evince 3.7.1. It seems, this bug can be closed as FIXED, because sidebar uses rounded selection rectangles as it was requested by  jorchube.
Comment 11 Germán Poo-Caamaño 2013-02-18 05:41:43 UTC
In http://git.gnome.org/browse/evince/commit/?id=1e88e7fd7 was introduced a custom cell renderer, which also fixes this bug.