Skip to content
This repository has been archived by the owner on Jan 26, 2022. It is now read-only.

Header templates: move group block margin to padding instead #325

Closed
wants to merge 3 commits into from

Conversation

kjellr
Copy link
Collaborator

@kjellr kjellr commented Jan 10, 2022

Fixes #272.

Since dynamic spacer block units will not be part of 5.9, this PR pursues an alternate fix for #272 that moves the unsupported margin values on the theme's dark headers to padding values on its (pre-existing) wrapper parent instead.

This is a little hard to find (since it's a variable, it currently shows up as an empty field in the editor), but it does allow the values to be editable. So it seems like a slight improvement at least.

Screenshot:

padding

@kjellr
Copy link
Collaborator Author

kjellr commented Jan 10, 2022

Closing in favor of #327.

@kjellr kjellr closed this Jan 10, 2022
@kjellr kjellr deleted the update/remove-margin branch January 10, 2022 18:35
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Empty gaps seen in the site editor.
1 participant