Issue #3353190 by ribel: Add Header top region to the Landing page template

[ribel]

Problem

Header top region is missing from overridden Landing Page template: page--node--landing-page.html.twig.

Solution

Add rendering of Header top region the same as in base template of socialblue theme: page--sky.html.twig:

{% if page.header_top and multi_line %}
  {{ page.header_top }}
{% endif %}

Issue tracker

• https://www.drupal.org/project/social/issues/3353190
• PROD-24944

Theme issue tracker

N/A

How to test

• Enable module Social Landing Page
• Go to theme settings
• Enable two-row header style
• Add some blocks to the Header top region
• Go to Landing page
• You should see block in the Header top region

Definition of done

Before merge

• Code/peer review is completed
• All commit messages are clear and clean. If applicable a rebase was performed
• All automated tests are green
• Functional/manual tests of the acceptance criteria are approved
• All acceptance criteria were met
• New features or changes to existing features are covered by tests, either unit (preferably) or behat
• Update path is tested. New hook_updates should respect update order, right naming convention and consider hook_post_update code
• Module can be safely uninstalled. Update/implement hook_uninstall and make sure that removed configuration or dependencies are removed/uninstalled
• This pull request has all required labels (team/type/priority)
• This pull request has a milestone
• This pull request has an assignee (if applicable)
• Any front end changes are tested on all major browsers
• New UI elements, or changes on UI elements are approved by the design team
• New features, or feature changes are approved by the product owner

After merge

• Code is tested on all branches that it has been cherry-picked
• Update hook number might need adjustment, make sure they have the correct order
• The Drupal.org ticket(s) are updated according to this pull request status

Screenshots

Before:

After:

Release notes

Header top region is now displayed on Landing Pages.

Change Record

N/A

Translations

N/A

[mergeable]

Thanks for contributing towards Open Social! A maintainer from the @goalgorilla/maintainers group might not review all changes from all teams/contributors. Please don't be discouraged if it takes a while. In the meantime, we have some automated checks running and it might be that you will see our comments with some tips or requests to speed up the review process. 😊

[Agamich-FE]

@ribel, I've checked if we have the header(header with two rows) on the landing page and everything looks good.

[ribel]

🍒 picked to 11.9.x and 11.8.x