Media 1202

[protitude]

Describe context / purpose for this PR

News Images - Rework

Issue link

#1202

Any other related PRs?

• https://github.com/UCBoulder/oit_dingo/pull/1215
• Media 1202 oit#16
• image: UCBoulder/oit_dingo#1203 sympa#3

Link to MultiDev instance

http://md-1202-ucbsub-oit.pantheonsite.io

Checklist for PR author

• I have checked that the PR is ready to be merged
• I have reviewed the DIFF and checked that the changes are as expected
• I have assigned myself or someone else to review the PR

[Copilot]

Pull request overview

Reworks the News “hero image” field block template to align with the theme’s Emulsify/BEM-based block markup patterns, likely as part of the broader “News Images - Rework” effort.

Changes:

• Wraps output in a content|render|trim emptiness check to avoid rendering an empty block container.
• Switches the wrapper markup to use bem() instead of attributes.addClass('pull-left').
• Updates the label heading to add the h2 class via title_attributes.

Comments suppressed due to low confidence (1)

templates/block/block--field-block--node--news--field-media-hero-image.html.twig:30

• bem() is being called with a first argument of 'block pull-left'. Elsewhere in the theme, additional classes are passed either as an array (e.g., bem(['no-list-style'])) or via the 4th extra argument. Passing a space-delimited string here can cause BEM class generation to treat pull-left as part of the base class and produce incorrect modifier classes (e.g., pull-left--<modifier>). Consider keeping the base class as 'block' and passing 'pull-left' as an extra class (or pass ['block', 'pull-left'] as the base).

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.