Issues after auto-update - WebAppManifest and Hero background image size

[tvercaut]

Hi,

Not sure if I am using wowchemy wrongly, I am really new to both hugo and wowchemy.

Today, when I started hugo server, I saw an automatic updated and got an error

hugo: downloading modules …
hugo: collected modules in 4528 ms
Error: from config: failed to resolve output format "WebAppManifest" from site config

After some searching I found this issue: gohugoio/hugo#7242
Applying the workaround there worked for me:

• Edit config/_default/config.toml to remove , "WebAppManifest"
• Call hugo mod get -u
• Edit config/_default/config.toml and add back `, "WebAppManifest"

But then I realised that my hero background image was displayed too big.

Uncommenting the image_size parameter in hero.md to cover seems to fix it but it seems weird I have to do taht as the comment mentions that cover is the default:

[design.background]
  # Background image.
  image = "test.jpeg"  # Name of image in `static/media/`.
  image_size = "cover"  #  Options are `cover` (default), `contain`, or `actual` size.

Am I using things wrongly?

Tom

[chrede88]

Looking at line 51 in the widget_page.html file (link: https://github.com/wowchemy/wowchemy-hugo-modules/blob/master/wowchemy/layouts/partials/widget_page.html#L51), that controls the background properties of the home page or other widget pages, it's clear that the image size is set by the background-size parameter. Now referring to this page (https://www.w3schools.com/csSref/css3_pr_background-size.asp), the default value of background-size is auto, which translates to actual in wowchemy. So it seems that the default value is not cover!

I'm not using the hero widget myself, but the hero.html widget partial is referring to a hero-media. Maybe it's better to that, instead of the background image option. Just a thought:)

[chrede88]

Yeah! That's was a little fast... Looking at this (4dd8f99), it seems Geo just fixed your issue;) And the default is indeed cover 😃