[Fixed]: Album list empty ablum cell glitch

[Matt Carlson]

IU noticed a glitch in the way Coppermine displays the album list. For empty cells it takes the spacer.gif and gives it a height equal to the setting for "Size of thumbnails in pixels". So if size of thumbnails in pixels is set at 120 and the only albums in that row are in landscape format (something like 4:3 format), the result will be extra space in the filled albums. Should not the empty cell just leave the spacer image at something like 1px for height and width?

My testing gallery
http://www.cornlines.com/gallery/

[Matt Carlson]

Solved / Fixed


Coppermine 1.4.19  - theme.inc.php - revison 4392

Line 465

Code Select Expand

<div class="thumbnails" style="background-color:transparent"><img src="images/spacer.gif" width="1" height="{SPACER}" border="0" class="image" style="border:0;margin-top:1px;margin-bottom:0" alt="" /></div>


Replace "{SPACER}" with a "1"

Now so as to keep the peace and follow instructions by developers one should not mess with the theme.inc.php but rather insert the code your own theme.php file as the section for "HTML template for the album list".

[Matt Carlson]

A screenshot of the issue before the change and after.

[Matt Carlson]

PS. Same issue exisits in the under-devolpement 1.5x version.

PPS. I wish there was a way to edit posts here - that or I am missing how to do so. 

[Joachim Müller]

Thanks for your report. The issue exists inside the definitions for $template_album_list_cat and $template_album_list. I have applied your suggested changes in the subversion repository both for cpg1.4.x as well as cpg1.5.x. This means that the fix will go into cpg1.4.19 and cpg1.5.0.

PPS. I wish there was a way to edit posts here - that or I am missing how to do so. 

You can't edit your postings due to abuse in the past. It's OK to reply with clarifications as you did.