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 773702 - "iTunes Podcast" page displays low resolution thumbnails.
"iTunes Podcast" page displays low resolution thumbnails.
Status: RESOLVED FIXED
Product: grilo
Classification: Other
Component: lua
unspecified
Other Linux
: Normal normal
: ---
Assigned To: grilo-maint
grilo-maint
Depends on:
Blocks:
 
 
Reported: 2016-10-30 16:03 UTC by gnome.vrb
Modified: 2016-10-31 13:57 UTC
See Also:
GNOME target: ---
GNOME version: ---

Attachments
Screenshot: Low resolution thumbnails. (827.54 KB, image/png)
2016-10-30 16:03 UTC, gnome.vrb 		  Details
Screenshot: High resolution thumbnails displayed for 'Apple Movie Trailers' (1.06 MB, image/png)
2016-10-30 16:05 UTC, gnome.vrb 		  Details
itunes-podcast: Sort the thumbnails in descending size (1.42 KB, patch)
2016-10-30 21:33 UTC, Bastien Nocera 		committed Details | Review

Description gnome.vrb 2016-10-30 16:03:36 UTC 
Created attachment 338796 [details]
Screenshot: Low resolution thumbnails.

iTunes provides good resolution thumbnails ( 100x100, 600x600 ). One of them can be used based on user configuration.
Comment 1 gnome.vrb 2016-10-30 16:05:35 UTC 
Created attachment 338797 [details]
Screenshot: High resolution thumbnails displayed for 'Apple Movie Trailers'

No issues with 'Apple Movie Trailers' page.
Comment 2 Bastien Nocera 2016-10-30 21:31:55 UTC 
There's no 600x600 thumbnails in the "browse" results, only in the search results. However we didn't sort the images (as mentioned in the source[1]). The attached patch fixes that.

[1]: https://git.gnome.org/browse/grilo-plugins/tree/src/lua-factory/sources/grl-itunes-podcast.lua?id=d521fa562508ccb51c924f35be85003e68364992#n156
Comment 3 Bastien Nocera 2016-10-30 21:33:23 UTC 
Created attachment 338804 [details] [review]
itunes-podcast: Sort the thumbnails in descending size

So that the first thumbnail is the biggest one.
Comment 4 gnome.vrb 2016-10-30 21:58:57 UTC 
That works.

Looks much better now. Thanks !
Comment 5 Victor Toso 2016-10-31 13:55:30 UTC 
Review of attachment 338804 [details] [review]:

sure
Comment 6 Bastien Nocera 2016-10-31 13:57:25 UTC 
Attachment 338804 [details] pushed as 83c0636 - itunes-podcast: Sort the thumbnails in descending size