Add logos block to content page #337
Conversation
This sounds like a perfect opportunity to get this onto |
|
@zerolab I sent Ben and Ian a short walkthrough of the design from my local machine. They approved it, so I think we can proceed to production. But I'm okay with testing on staging first if that's your preference. |
| @@ -4,11 +4,11 @@ | |||
| position: relative; | |||
|
|
|||
| &__heading { | |||
| margin-bottom: 20px; | |||
| margin-bottom: 80px; | |||
There was a problem hiding this comment.
We might want to reduce this, and maybe increase the margin below the logos instead? Feels a bit weird that the bottom margin of the heading is larger (80px) than the bottom margin between the logos and the next heading (50px), which is a separate group.
There was a problem hiding this comment.
Submitted a commit that sets the margin to 60px for all views. Please let me know if there's anything else I should fix.
We need some small adjustments to the Logos block and to the blocks available on the Content page to create a suitable MVP design for the "Who Uses Wagtail" page.
This PR makes the Logos block available on the Content page type and provides an optional heading that can be used to organise the logos.
Ben E and Ian B from Torchbox want to review the design a bit before we proceed with merging, so I'm submitting this PR as a draft for now.