Hero Pattern: Incorrect section placement

[jmuzina]

Describe the bug

While building canonical/canonical.com#1406, I noticed an issue with the hero macro that prevented the page from being built with the hero macro. When flipping between my in-progress support page and the postgresql managed page, I found that the spacing between the title and the description was different.

Currently, the title blocks are wrapped in .p-section--shallow, and the description is wrapped in a separate .p-section--shallow. This creates a lot of space between the title area and the description:

Following the example from the postgresql managed page, the titles and description can all be wrapped in a single shallow section to tighten things up a bit:

[syncronize-issues-to-jira]

Thank you for reporting us your feedback!

The internal ticket has been created: https://warthogs.atlassian.net/browse/WD-16099.

This message was autogenerated

[jmuzina]

It's not quite as simple as just changing to wrapping both in a shallow section, as the heading and subheading are not always in the same container. The full-width image version splits these across two separate columns. We should make sure that the correct element has shallow section spacing in that case.