Album Art Alignment

[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



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

V4.5 on WHS R1

[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.

[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!

[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