payloadcms / website

The official Next.js website for payloadcms.com
https://payloadcms.com
MIT License
372 stars 100 forks source link

feat: update default hero #308

Closed paulpopus closed 9 months ago

paulpopus commented 9 months ago

Old: image

image

New: image image

PatrikKozak commented 9 months ago

@paulpopus The spacing between the breadcrumbs and the heading seems too large. I'd suggested removing the top and bottom margin on the breadcrumb itself - by targeting it through the default hero though

Feel free to adjust the spacing differently if you think there is a better way

Screen Shot 2024-01-30 at 11 26 04 AM

PatrikKozak commented 9 months ago

@paulpopus - is this white box in the design meant to be media for this default hero or no? What is it?

Screen Shot 2024-01-30 at 11 34 07 AM

PatrikKozak commented 9 months ago

@paulpopus - Do you see how the content on the right seems to sit higher up than the heading? Design doesn't show this to be the case.

Screen Shot 2024-01-30 at 11 29 03 AM

paulpopus commented 9 months ago

@paulpopus - is this white box in the design meant to be media for this default hero or no? What is it?

Just placeholder