hugo-theme-bleak icon indicating copy to clipboard operation
hugo-theme-bleak copied to clipboard

Published 20 hours ago • verified publisher icon Zenithar

Hardcoded low-resolution cover image URL

Open bstegmaier opened this issue 8 years ago • 2 comments

Summary The low resolution cover image is not configurable and hard-coded as /images/cover.low.jpg.

Steps to reproduce

  • Remove any static/images/cover* files
  • Add an image static/images/any_name_other_than_cover.jpg
  • Configure it to be the cover image:
   cover = "/images/any_name_other_than_cover.jpg"
  • Serve and browse the site

Result Chrome console complains about a non-existing /images/cover/low.jpg file.

Suggested fix How about making the low res cover configurable via e.g. .Site.Params.coverLow? This is the relevant line: https://github.com/Zenithar/hugo-theme-bleak/blob/a8dcb798ae7f98ace354f42c3a23b40fa2c54542/layouts/index.html#L13

bstegmaier avatar Apr 16 '17 18:04 bstegmaier

PR are welcome ^^

Zenithar avatar Apr 17 '17 16:04 Zenithar

Sounds like a great idea. Basically making the low res version of the cover image an optional feature.

curtiscde avatar Aug 23 '17 10:08 curtiscde