From 96f3de022362dbb7ea4d80eb5f281e4a51d0b4ab Mon Sep 17 00:00:00 2001 From: Christoph Haas Date: Fri, 24 Apr 2015 17:54:37 +0200 Subject: [PATCH] Show placeholder image if a package has no screenshots. --- app/views/packages/details.slim | 27 ++++++++++++++------------- 1 file changed, 14 insertions(+), 13 deletions(-) diff --git a/app/views/packages/details.slim b/app/views/packages/details.slim index b6f0bcc..817766b 100644 --- a/app/views/packages/details.slim +++ b/app/views/packages/details.slim @@ -11,19 +11,20 @@ .small-7.medium-7.columns p.subtitle = @package.description - - - @package.screenshots.each do |screenshot| - .row.listview - .small-12.columns - a.black.fancybox href=screenshot.image.url(:large, timestamp: false) rel='fancybox-thumb' title=screenshot.caption - = image_tag(screenshot.image.url(:large, timestamp: false), alt: screenshot.caption) - .imgcaption =screenshot.caption - // TODO: Enable button only if admin or uploader - a.button.tiny.alert[ - href=delete_screenshot_path(@package.name, screenshot.id) - onclick="return confirm('Really delete the screenshot?');" - ] Delete screenshot - + - if @package.screenshots.count > 0 + - @package.screenshots.each do |screenshot| + .row.listview + .small-12.columns + a.black.fancybox href=screenshot.image.url(:large, timestamp: false) rel='fancybox-thumb' title=screenshot.caption + = image_tag(screenshot.image.url(:large, timestamp: false), alt: screenshot.caption) + .imgcaption =screenshot.caption + // TODO: Enable button only if admin or uploader + a.button.tiny.alert[ + href=delete_screenshot_path(@package.name, screenshot.id) + onclick="return confirm('Really delete the screenshot?');" + ] Delete screenshot + - else + img src='/images/dummy/no-screenshots-upload-one.svg' width='100%' // TODO: Enable comments in a later version // = partial '/package/comments'