Come together with the global Drupal community in Rotterdam, 28 Sept – 1 Oct 2026. Sessions, contribution, connection, and Early Bird savings until 8 June.Problem/Motivation
Follow-up to:
#3511877: Add SDDS global header styling
In that issue, a header_panel was introduced that has a soft border and two split slots.
We have generic components (e.g. container, columns, flexible slot) that were improved in this issue:
#3510427: Add SDDS global footer styling
It would be nice to move the styling needed for the header menu area into these generic components somehow.
Steps to reproduce
Proposed resolution
TBD how they would be integrated into the existing components.
Remaining tasks
- Figure out best way to integrate
- Update code including removing header_panel
- Review/test/commit
User interface changes
The header_panel won't show up but the equivalent features would be available on the generic components.
API changes
Data model changes
| Comment | File | Size | Author |
|---|---|---|---|
| #10 | sdds-social-media-footer-testing.png | 435.42 KB | kristen pol |
Issue fork demo_design_system-3513066
Show commands
Start within a Git clone of the project using the version control instructions.
Or, if you do not have SSH keys set up on git.drupalcode.org:
changes
Comments
Comment #2
kristen polNote:
#3513270: Twig\Error\LoaderError: Template "@atoms/starshot-menu/starshot-menu.twig" is not defined
Comment #3
kristen polNote the split component functionality that we'd like in the footer styling should be considered when doing this one too.
#3510427: Add SDDS global footer styling
Comment #6
kristen polCool! I'll take a look :)
Note that Lionel and I talked about this one yesterday and the plan is to keep the header_panel but add the styles from it to the generic components to show both ways of building the header.
Comment #7
kristen polAh... the social media stuff got into this MR... hmm... I'll still just try to test just the columns+container.
Comment #8
kristen polHad a call with Lionel and all the social media changes are going to happen in this MR. He's doing a bit of cleanup right now.
Comment #9
kristen polThanks! I'll review.
Comment #10
kristen polI've done some minor name tweaks and tested and this is working pretty well... I had some layout issues but maybe my settings aren't quite right.
This is good enough to merge and we can make a release for Pam.
Then we can do any minor tweaks in another issue.
Comment #12
kristen polThanks!
Comment #13
kristen polRecording of testing:
https://youtu.be/5l7gxrf764k
Comment #14
kristen polupdate parent