Following fix on MBS-4465, see http://chatlogs.musicbrainz.org/musicbrainz-devel/2012/2012-09/2012-09-04.html#T18-01-40-504975
Question: currently every call to display_artwork adds the @rel; is there anywhere else we shouldn't be? I'm not sure how useful "page through all the images on this page of edits" is (but it's not obviously wrong); "page through all the covers within this collection" seems like it should stay... not sure what else.
I'd say: don't include @rel anywhere, excepted for images on the release/cover-art page (minus the one in the sidebar of course).
How does that sounds?
Hmm... I didn't see you've mentioned collection... but I'm not sure it would really be useful there.
I think the original intent was mainly "page through all the images of one release".
The stuff that displays it for collections isn't even shipped yet AFAIK, worth noting. But I'm fine with only including @rel on the release coverart page for now and seeing if people want it elsewhere; you're right about the original intent, I was just wondering if it'd be possible to pre-empt some tickets by putting it more places than just there
Actually, here's one that may be helpful: the reorder cover art page. Thoughts?
Indeed, this one make sense too. Good catch!
Wouldn't it be easier to keep putting @rel on everything, but changing the selector to require all elements to be in some sort of 'grouping container'?
ocharles: thickbox groups by what has a shared rel attribute. In general I think it's probably not a good idea to be screwing with our third-party code (except for the bugfix I had to make), meaning that this is the way to do it.