• Type: Improvement
• Resolution: Fixed
• Priority: Normal
• Fix Version/s: None
• Affects Version/s: None
• Component/s: Cover Art
• Labels:

  None

[PICARD-1189] Support for CAA 1200 px thumbnails

Collapse comment: Zas added a comment - 2018-02-13 15:26

Zas added a comment - 2018-02-13 15:26

obtext https://github.com/metabrainz/picard/pull/838 should fix the issue, thanks for testing and report

Expand comment: Zas added a comment - 2018-02-13 15:26

Zas added a comment - 2018-02-13 15:26 obtext https://github.com/metabrainz/picard/pull/838 should fix the issue, thanks for testing and report

Collapse comment: Alec Edgington added a comment - 2018-02-11 16:52

Alec Edgington added a comment - 2018-02-11 16:52

I noticed something strange after rebuilding with this patch. I have never touched this config setting in Picard, so it was set to '500px', the default. But with the new build, Picard started downloading maximum-resolution images all the time. The config UI still showed '500px' checked, but when I changed this to '250px' and then back to '500px', behaviour returned to how it was before, and only 500px images were downloaded (as desired).

Unless there's some explanation particular to my set-up this could cause a lot of confusion.

Expand comment: Alec Edgington added a comment - 2018-02-11 16:52

Alec Edgington added a comment - 2018-02-11 16:52 I noticed something strange after rebuilding with this patch. I have never touched this config setting in Picard, so it was set to '500px', the default. But with the new build, Picard started downloading maximum-resolution images all the time. The config UI still showed '500px' checked, but when I changed this to '250px' and then back to '500px', behaviour returned to how it was before, and only 500px images were downloaded (as desired). Unless there's some explanation particular to my set-up this could cause a lot of confusion.

Collapse comment: vishal choudhary added a comment - 2018-02-10 17:17

vishal choudhary added a comment - 2018-02-10 17:17

https://github.com/metabrainz/picard/pull/833 merged.

Expand comment: vishal choudhary added a comment - 2018-02-10 17:17

vishal choudhary added a comment - 2018-02-10 17:17 https://github.com/metabrainz/picard/pull/833 merged.