While I was testing another patch I had to switch off css aggregation. Then I noticed an error in the broser console

Refused to apply style from 'http://umami.lndo.site/core/profiles/demo_umami/themes/umami/css/compone...' because its MIME type ('text/html') is not a supported stylesheet MIME type, and strict MIME checking is enabled.

Further investigation let to the conclusion that this file is missing.

If not needed, proposed solution would be to remove this file from the umami.library.yml

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

parijke created an issue. See original summary.

parijke’s picture

here's a mini patch if not needed

parijke’s picture

Status: Needs work » Needs review
Eli-T’s picture

Looks like this was deliberately removed in http://cgit.drupalcode.org/drupal/commit/?id=9949db9f0581806185cc90b96eb... but not from the library

googletorp’s picture

Status: Needs review » Reviewed & tested by the community

Good catch

parijke’s picture

All other files mentioned in umami.library.yml sre actually there, so article.css is the only one missing

longwave’s picture

I wonder if it is worth adding a warning message to core if it detects a missing file declared in a theme (or module, I guess).

googletorp’s picture

Re #7 If we want to do so, that should be a separate issue. Don't think this is a general issue though so not sure if that would be worth it.

Eli-T’s picture

+1 for not doing that in this issue

longwave’s picture

Oh sure, I wasn't suggesting doing it here, just that if people think it is worthwhile we could open a new issue, as if we can make this mistake in a core theme I am sure it has happened in contrib/custom themes as well.

lauriii’s picture

Title: article.css seems to be missing » Remove non-existing article.css from the libraries.yml

  • lauriii committed d938a14 on 8.7.x
    Issue #3001893 by parijke, Eli-T: Remove non-existing article.css from...
lauriii’s picture

Status: Reviewed & tested by the community » Fixed

Committed d938a14 and pushed to 8.7.x. Also cherry-picked to 8.6.x. Thanks! 🚀

  • lauriii committed 49b77a9 on 8.6.x
    Issue #3001893 by parijke, Eli-T: Remove non-existing article.css from...

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.