Drupal uses-responsize-images unclear

[exterkamp]

The second half of the string https://github.com/GoogleChrome/lighthouse-stack-packs/blob/master/packs/drupal.js#L28 is unclear.

Use the Responsive Image Styles when rendering image fields through view modes, views, or images uploaded through the WYSIWYG editor.',

Are there three ways of rendering an image field? (through a view mode, a view, or via upload from the WYSIWYG editor) Or is the WYSIWYG editor different and is uploading images directly instead of through fields?

This needs to be re-written by someone who knows what all these Drupal terms mean 😄

[paulirish]

full thing:

Ensure that you are using the native Responsive Image Styles provided from Drupal (available in Drupal 8 and above). Use the Responsive Image Styles when rendering image fields through view modes, views, or images uploaded through the WYSIWYG editor.

cc @moerazem

cc @webchick maybe you can help us out with wording or tag in someone who can?

this text is shown to lighthouse users when analyzing a page that's hosted by drupal. so they should have some familiarity. Perhaps we can remove some of the detail here? does https://www.drupal.org/docs/mobile-guide/responsive-images capture most of what they need to know?

[moerazem]

I'd say the most common ways of rending images in Drupal are indeed view modes, views, or WYSIWYG embeds.
This terminology should be clear to Drupal developers/site builders.

The reason I wanted to emphasize those three ways, is because it's very common for Drupal developers/site builders to configure Responsive Image Styles on a node/content when displayed in its own page, but skip that (or miss it) when displaying it via a "view". "Views" can also display images through individual "fields" or a "view mode" that includes an image.
Hence, this description to emphasize those various ways to alert Drupal developers/site builders when rendering images.

Any edit/suggestion to make it clearer is also extremely appreciated.