-
Notifications
You must be signed in to change notification settings - Fork 5k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Change the footer navigation #12376
Comments
|
Thank you. I created a Figma file to test it on top of the Footer we have for the design system > feel free to edit and comment My first reaction is that it can be too many columns. I tried to remove one, but even so... might be too much maybe add a dipper level like the header? accordion like the mobile menu? or curate some of the links and make the list smaller |
Thanks, this is a great visualization. I didn't know that the previous version was not good, so I shouldn't simulate that with new content. I have curated the list to try with a shorter version. If it is workable, for example we can call Learn hub based on the main content that we find there = What is Ethereum, ETH & web3? Maybe a bit long, but 1 row instead of 4. (What is Ethereum & ETH? also a good version) Same for Community hub = Online communities & events We could maybe merge the columns like Learn & Use, Build & Research, Participate (also with the items from General). One option is 3 columns with long list, or again a shorter curated list like below. |
Hey! I noticed there is a YouTube icon in your prototype, so I added a YouTube link and a new Twitter icon in pull request #12382. If you or the team do not plan to update the Twitter icon, I can remove it, leaving only the YouTube icon.
|
Oh, you just found a bug on the design system, the YouTube icon was removed last year, and I did not update the Figma file, my bad, sorry about this (was this PR #10775) |
I'm really sorry. I can directly modify and remove the YouTube icon in the pull request, leaving only a Twitter x icon. You can check it in #12382. |
Thank you for the second version, but I think it makes harder to understand I did some explorations on figma, but I got to a point that the quick win here is your first proposal with minor design adjustments.
@Satglow let me know what do think on figma > https://www.figma.com/file/cKWwsQVF762HM7IuSnH8xn/Footer-links-update?type=design&node-id=4%3A43546&mode=design&t=rReoPKKv7zMDOBfh-1 @wackerow what do think, is it fast to build a PR with is to test on the browser? |
Thanks a lot, I am okey with that! The shorter list still can work without merging the items, if you are looking for less item. Like this: I think the 'secondary' is great as footer of footer, just we should not leave out, because this is not available elsewhere - i think this is one of the main purpose of the footer part. |
Is your feature request related to a problem? Please describe.
With the new Header Navigation in production, we want to explore the change on the footer.
It does not need to have the same information architecture, but it should be updated.
Describe the solution you'd like
A new navigation on the footer
Describe alternatives you've considered
as many as possible
Additional context
No response
Would you like to work on this issue?
The text was updated successfully, but these errors were encountered: