Skip to content

Commit

Permalink
Merge branch 'picwall-direct-link' into next
Browse files Browse the repository at this point in the history
  • Loading branch information
nodiscc committed Mar 15, 2015
2 parents aa22244 + 4a1a119 commit b4b7d33
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 2 deletions.
1 change: 0 additions & 1 deletion index.php
Original file line number Diff line number Diff line change
Expand Up @@ -1271,7 +1271,6 @@ function renderPage()
if ($thumb!='') // Only output links which have a thumbnail.
{
$link['thumbnail']=$thumb; // Thumbnail HTML code.
$link['permalink']=$permalink;
$linksToDisplay[]=$link; // Add to array.
}
}
Expand Down
2 changes: 1 addition & 1 deletion tpl/picwall.html
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,7 @@
<div id="picwall_container">
{loop="linksToDisplay"}
<div class="picwall_pictureframe">
{$value.thumbnail}<a href="{$value.permalink}"><span class="info">{$value.title|htmlspecialchars}</span></a>
{$value.thumbnail}<a href="{$value.url}"><span class="info">{$value.title|htmlspecialchars}</span></a>
</div>
{/loop}
</div>
Expand Down

1 comment on commit b4b7d33

@alexisju
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Je n'aime pas vraiment cette idée. Le picwall n'affiche que des petits thumbnails. Il n'y a pas la place pour les commentaires ou d'autres infos (tag, liens,...). Le span info est très limité aussi.
Si l'url est corrompue, c'est vraiment difficile de pouvoir mettre à jour le l'entrée si l'image renvoie à autre chose que le permalink.

Je remets un permalink sur les images du picwall de mon template (avec {$value.linkdate|smallHash})
Je pense qu'on devrait envisager de revenir en arrière sur ce point.

(Et pour les templates, en général, c'est dommage d'avoir perdu la possibilité d'utiliser {$value.permalink} )

Please sign in to comment.