Skip to content

Add new docs footer#33453

Merged
mdo merged 2 commits intomainfrom
v5-new-footer
Mar 31, 2021
Merged

Add new docs footer#33453
mdo merged 2 commits intomainfrom
v5-new-footer

Conversation

@mdo
Copy link
Copy Markdown
Member

@mdo mdo commented Mar 23, 2021

Screen Shot 2021-03-23 at 3 37 28 PM

Replaces the tiny footer with something more representative of the community projects, resources, and links.

Preview: https://deploy-preview-33453--twbs-bootstrap.netlify.app/

@mdo mdo requested a review from a team as a code owner March 23, 2021 22:38
@XhmikosR
Copy link
Copy Markdown
Member

XhmikosR commented Mar 24, 2021

I like it! One thought, though, perhaps using 5 is too much empty space? Maybe try with pt-4, mt-4, py-4 and see how it looks?

EDIT:

Oh, also, the footer links don't really make out here

image

Maybe a little bolder (font-weight: 600;) should do it:

image

EDIT2: Not sure about still using .small either.

@ffoodd
Copy link
Copy Markdown
Contributor

ffoodd commented Mar 24, 2021

Very nice!

@mdo
Copy link
Copy Markdown
Member Author

mdo commented Mar 31, 2021

Liking the space and contrast for now on links—they should be very subdued for this. Happy to iterate later though.

@mdo mdo merged commit 6e7f1a9 into main Mar 31, 2021
@mdo mdo deleted the v5-new-footer branch March 31, 2021 06:08
@XhmikosR
Copy link
Copy Markdown
Member

noopener is probably useless without target="_blank" so this should be addressed for sure.

Also, I do think it'd be better if the footer links were moved in data later and I definitely think the space is just too much.

Just my 2 cents :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants