Repository navigation
Add spacing between the mobile fixed header and the page title (#763) - #953
Conversation
jjroelofs
left a comment
There was a problem hiding this comment.
Reviewed commit dafa380 against 8.x. No actionable findings.
The selector adds the requested spacing for mobile fixed headers without an admin toolbar. The existing toolbar-specific rules remain intact, and the build correctly converts the added 16px to 1rem. The other compiled CSS changes use equivalent color notation for the existing shadows.
Validation: SCSS stylelint passed locally. Recompiled the changed SCSS with Sass, Autoprefixer, and the repository's px-to-rem settings; all 147 CSS declarations match the committed CSS after normalizing those equivalent colors. All five reported CI checks passed. Reviewed the surrounding header styles, template, and JavaScript; a live Drupal/browser visual check was not run.
Linked issues
Solution
With the mobile fixed header on and no admin toolbar, the body padding equals the header height, so the page title starts at the header edge. The padding now adds 16px (1rem in the compiled CSS) of space. Logged-in users with a toolbar are unchanged. Compiled CSS rebuilt with the Docker build.
Checklist