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 478363 - Missing thumbnail frames
Missing thumbnail frames
Status: RESOLVED FIXED
Product: nautilus
Classification: Core
Component: Thumbnails
2.20.x
Other Linux
: Normal normal
: ---
Assigned To: Christian Neumair
Nautilus Maintainers
Depends on:
Blocks:
 
 
Reported: 2007-09-19 14:18 UTC by Michael Kuhn
Modified: 2008-01-18 19:30 UTC
See Also:
GNOME target: ---
GNOME version: 2.19/2.20


Attachments
quick patch (919 bytes, patch)
2007-09-20 20:43 UTC, Christian Neumair
none Details | Review

Description Michael Kuhn 2007-09-19 14:18:27 UTC
Nautilus 2.20.0 does not show thumbnail frames anymore. However: if I do a “touch $image.jpg” and refresh the Nautilus view (Ctrl-R), the new thumbnail has a frame.
After a Nautilus restart (killall nautilus) the frame is gone again.
I already tried a “rm -rf ~/.thumbnails”, but it did not help.

I'm pretty sure this worked correctly in 2.19.91.
Comment 1 Marcus Klemm 2007-09-19 16:28:46 UTC
I can confirm this bug. It came with 2.20.
Comment 2 Patrick Ohearn 2007-09-20 12:11:51 UTC
Second confirmation here with 2.20.
Comment 3 Alexander Larsson 2007-09-20 13:14:12 UTC
Christian, could you look at this. Its likely from the async thumbnail loading patch.
Comment 4 Christian Neumair 2007-09-20 20:43:05 UTC
Looks like the asynchronous loading function should also add a frame for thumbnails if they don't have any alpha channels, similar to its synchronous variant nautilus-icon-factory.c:load_icon_file().

I'll attach a quick patch, it doesn't look very pretty. I didn't want to change nautilus_thumbnail_frame_image semantics, I hope it's OK.

Sorry that I didn't note this during hacking. I usually install the development version into a prefix other than /usr, and just install the src/ contents, so I'm used to lack of thumbnail frames and didn't consider it. 
Comment 5 Christian Neumair 2007-09-20 20:43:38 UTC
Created attachment 95921 [details] [review]
quick patch
Comment 6 Laurento Frittella 2007-09-22 09:55:15 UTC
Thanks Christian your patch seems to work.
Comment 7 Alexander Larsson 2007-09-26 07:06:26 UTC
Looks good. Please commit (to head and 2.20 branch)
Comment 8 Sebastien Bacher 2007-09-26 14:05:12 UTC
note that the async thumbnailing also create bug #480608
Comment 9 Christian Neumair 2007-09-27 17:21:53 UTC
Committed, closing.
Comment 10 George Vlahavas 2008-01-18 19:30:18 UTC
I've applied the patch to nautilus 2.20.0 but I still have the problem described above.

For example: if I take a screenshot (png) with gnome-screenshot, it will never get a border. If I do

convert screenshot.png screenshot.jpg

the jpg file gets a border
and if I then do

convert screenshot.jpg screenshot2.png

the new png file will get a border!