MWPW-157888 - Hero marquee w/ no-min-height and no l/r padding when using a full-width variant

[ryanmparrish]

As and author in a milo project using a [hero-marquee]

1. I want to use this block with out a minimal height so the block has a natural height based on content.
2. I want to use this block in an ups grid w/ the variant full-width and not have the extra left/right padding seen in this scenario.

Example screenshot - Notice the block height is huge, the l/r padding is causing the block to go outside its bounds.

---

Fixed

1. Add a no-min-height variant to the hero-marquee.
2. Address padding issue on hero-block. Erroneously comes from the has-bg class being added.

Resolves: MWPW-157888

Test URLs:

• Before: https://main--milo--adobecom.hlx.page/drafts/steenmeyer/hero-editorial-layout-pattern-examples/hero-editorial-no-min-height-masonry1?martech=off
• After: https://rparrish-hero-no-min--milo--adobecom.hlx.page/drafts/steenmeyer/hero-editorial-layout-pattern-examples/hero-editorial-no-min-height-masonry1?martech=off

[aem-code-sync]

Hello, I'm the AEM Code Sync Bot and I will run some actions to deploy your branch and validate page speed.
In case there are problems, just click a checkbox below to rerun the respective action.

• Re-run PSI checks
• Re-sync branch

Commits

• 28ae72a ✅ (latest)
• 80085df ✅

[aem-code-sync]

	Page	Scores	Audits	Google
M	/drafts/steenmeyer/hero-editorial-layout-pattern-examples/hero-editorial-no-min- height-masonry1?martech=off
D	/drafts/steenmeyer/hero-editorial-layout-pattern-examples/hero-editorial-no-min- height-masonry1?martech=off

[ryanmparrish]

Adding GWP drafts here - They were negatively impacting the PSI check

GWP drafts:

• https://rparrish-hero-no-min--milo--adobecom.hlx.page/drafts/steenmeyer/hero-editorial-layout-pattern-examples/hero-editorial-no-min-height-masonry - layout pattern = hero marquee block and editorial cards using masonry grid
• https://rparrish-hero-no-min-milo-adobecom.hlx.page/drafts/steenmeyer/hero-editorial-layout-patte[…]sectionmeta-masonry-grid?nocache=1724957601974 - layout pattern = hero-marquee block is it's own section editorial cards using masonry grid
• https://rparrish-hero-no-min-milo-adobecom.hlx.page/drafts/steenmeyer/hero-editorial-layout-patte[…]-height-sectionmeta-4-up?nocache=1724956950870 - layout pattern = hero-marquee block is it's own section editorial cards using 4 up grid

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 96.10%. Comparing base (d90dcc0) to head (28ae72a).
Report is 70 commits behind head on stage.

Additional details and impacted files

@@            Coverage Diff             @@
##            stage    #2861      +/-   ##
==========================================
+ Coverage   95.90%   96.10%   +0.19%     
==========================================
  Files         173      215      +42     
  Lines       45842    53956    +8114     
==========================================
+ Hits        43967    51856    +7889     
- Misses       1875     2100     +225     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[SilviuLCF]

Verified by @NadiiaSokolova and ready to STAGE from QA perspective

Testing details can be found here
https://jira.corp.adobe.com/browse/MWPW-157888

[milo-pr-merge]

Skipped 2861: "MWPW-157888 - Hero marquee w/ no-min-height and no l/r padding when using a full-width variant" due to file "libs/blocks/hero-marquee/hero-marquee.js" overlap. Merging will be attempted in the next batch