Issue Details (XML | Word | Printable)

Key: MBS-5447
Type: Improvement Improvement
Status: Closed Closed
Resolution: Fixed
Priority: Normal Normal
Assignee: Ian McEwen
Reporter: PATATE12
Votes: 2
Watchers: 2
Operations

If you were logged in you would be able to see more operations.
MusicBrainz Server

Bandwidth performance: revert MBS-4764 now that thumbnail quality has improved

Created: 11/Oct/12 01:15 PM   Updated: 28/Jan/13 10:15 AM   Resolved: 28/Jan/13 10:15 AM
Component/s: Cover art, Data display, Edit system, Templates, User interface
Affects Version/s: None
Fix Version/s: 2013-01-28

Issue Links:
Relates
 


 Description  « Hide

WOW MY GODS, Add cover art edits are displaying 500×500px thumbnails in a 250×250px sized box (it seems to be the case of all types of CAA edits BTW).

Those thumb500.jpg are usually three times bigger (in octets) than the thumb250.jpg that we should be using.
By using the thumb250.jpg (remember that we display them in a square 250 box anyway) we could improve edit search loading times and bandwidth usage greatly (and it is heavily needed).

☞ Without saying that, moreover being useless, displaying a client resized picture is usually uglier than displaying genuine size (be it server resized).



Sort Order: Ascending order - Click to sort in descending order
Ian McEwen added a comment - 11/Oct/12 11:53 PM

This is intentional: MBS-4674


PATATE12 added a comment - 12/Oct/12 08:07 AM

I re-open because, as you can see, there is no quality issue now.
The 500px image displayed in 250px box is not better looking than the 250px picture.

For me MBS-4674's problem is solved now (I have never complained anyway) and we should do things properly, without loading unused data and slowing down like this.


Aurélien Mino added a comment - 12/Oct/12 08:15 AM

I agree with jesus2099 (aka patate12): maybe 250px image doesn't have the better quality, but browser downscaling of 500px to 250px is usually not very good either.

So since the final quality is not really different, saving bandwidth (for both MB and users) seems to be a good idea.


Ian McEwen added a comment - 06/Jan/13 02:38 AM

Amending summary to be more descriptive


Ian McEwen added a comment - 06/Jan/13 02:39 AM