Adding attributes to views-view-list.html.twig doesn't work if Views List class Style option is empty

Can confirm that this patch does not apply, but is absolutely still needed.

I have attached a patch for same, please review

Next step would be to get a test case to show the issue.

At the very least I can say that patch #24 does apply and does fix the issue. Tested on Drupal 9.5.4

Merge request !13162 is a straightforward reroll— exactly what is in daniel korte's patch initial patch in #2 🐛 Adding attributes to views-view-list.html.twig doesn't work if Views List class Style option is empty Needs work got into core through changes made out of this issue (without tests?!), so technically the problem stated in the title of this issue is now fixed in Drupal core. My re-roll of a merge request therefore only preserves jwilson's proposal in #12 🐛 Adding attributes to views-view-list.html.twig doesn't work if Views List class Style option is empty Needs work , implemented in gauravvvv's patch in #24 🐛 Adding attributes to views-view-list.html.twig doesn't work if Views List class Style option is empty Needs work , to not blow away attributes that were added by a possibly earlier preprocess.

I agree this is how it should work, but am not up for a test, let alone making this consistent everywhere, and i know there are rumblings about somehow getting rid of hook_preprocess entirely, however that would work, so i leave this humble re-roll and update as my offering.

Thanks for re-rolling!

Tagging for issue summary as the standard template appears to be missing.

Based on the title seems like something that tests should cover so am leaving that tag.

Thanks!