Page 1 of 1

Album Art Alignment

PostPosted: Fri Aug 19, 2011 2:12 pm
by BKKKPewsey
I don't know if it just me but the alignment of album art sometimes is a bit "off".
I didn't notice this on the last beta and only seems to affect certain album groups :?

Image

This is the same in both chrome and FF so unlikely to be browser issue but
it could be me lappie which is now getting a bit ancient (like owner) but whilst I can still get valves for it I'll keep it :lol:

V4.5 on WHS R1

Re: Album Art Alignment

PostPosted: Wed Dec 21, 2011 8:16 pm
by ytechie
BKKKPewsey, I have the same issue when the cover art size is set to anything less than Large in the player settings. I looked at the jsp files line by line for hours, and I couldn't figure this one out.

Re: Album Art Alignment

PostPosted: Sun Jan 15, 2012 1:46 am
by ytechie
I actually modded my instance of subsonic to prevent this issue. With the mod, it only shows the album title under the cover art if the cover art size is set to large. Otherwise it just shows a zoom link which zooms the image from the artist view. Here is the mod:

We are going to edit the main.jsp. Here is the part we are going to edit:

Code: Select all
<td style="vertical-align:top; width:100%">
   <c:forEach items="${model.coverArts}" var="coverArt" varStatus="loopStatus">
      <div style="float:left; padding:5px">
         <c:import url="coverArt.jsp">
            <c:param name="albumPath" value="${coverArt.parentFile.path}"/>
            <c:param name="albumName" value="${coverArt.parentFile.name}"/>
            <c:param name="coverArtSize" value="${model.coverArtSize}"/>
            <c:param name="coverArtPath" value="${coverArt.path}"/>
            <c:param name="showLink" value="${coverArt.parentFile.path ne model.dir.path}"/>
            <c:param name="showZoom" value="${coverArt.parentFile.path eq model.dir.path}"/>
            <c:param name="showChange" value="${(coverArt.parentFile.path eq model.dir.path) and model.user.coverArtRole}"/>
            <c:param name="showCaption" value="true"/>
            <c:param name="appearAfter" value="${loopStatus.count * 30}"/>
         </c:import>
      </div>
   </c:forEach>


We are going to add " or model.coverArtSize < 150" to the end of the zoom parameter. The final result loks like this:

Code: Select all
<td style="vertical-align:top; width:100%">
   <c:forEach items="${model.coverArts}" var="coverArt" varStatus="loopStatus">
      <div style="float:left; padding:5px">
         <c:import url="coverArt.jsp">
            <c:param name="albumPath" value="${coverArt.parentFile.path}"/>
            <c:param name="albumName" value="${coverArt.parentFile.name}"/>
            <c:param name="coverArtSize" value="${model.coverArtSize}"/>
            <c:param name="coverArtPath" value="${coverArt.path}"/>
            <c:param name="showLink" value="${coverArt.parentFile.path ne model.dir.path}"/>
            <c:param name="showZoom" value="${coverArt.parentFile.path eq model.dir.path or model.coverArtSize < 150}"/>
            <c:param name="showChange" value="${(coverArt.parentFile.path eq model.dir.path) and model.user.coverArtRole}"/>
            <c:param name="showCaption" value="true"/>
            <c:param name="appearAfter" value="${loopStatus.count * 30}"/>
         </c:import>
      </div>
   </c:forEach>


Hope this helps! :D

Re: Album Art Alignment

PostPosted: Sat Feb 18, 2012 5:50 pm
by BKKKPewsey
2 things I would do
  1. Delete jetty cache - \subsonic\jetty\xxxx\jsp\org\apache\jsp\Delete what's here (Stop/restart SS)
  2. Delete browser cache

:mrgreen: