Skip to content
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

Remove page links from footer and move to front page #188

Open
agjohnson opened this issue Mar 20, 2023 · 1 comment
Open

Remove page links from footer and move to front page #188

agjohnson opened this issue Mar 20, 2023 · 1 comment
Assignees
Labels
Improvement Minor improvement to code

Comments

@agjohnson
Copy link
Contributor

The front page has a "Resources" section that would be a better place to mock up navigation structure. Currently we're using the footer for each page we add, which is rather hidden and pollutes the footer more. I'm avoiding overloading the footer so that that website footer and application footer can match.

image

Ultimately, we should have a top-level page for comparisons and tools. But for now, the front page resources section could be the start of this, and both sections could be cards in that "Resources" section.

@agjohnson agjohnson added the Improvement Minor improvement to code label Mar 20, 2023
@ericholscher
Copy link
Member

ericholscher commented Mar 20, 2023

I think we want links here to be subtle, but I agree the footer is not the right place. But these are landing pages for Google, not really for users, so I think they make sense to link from the footer for now, and have an index if we need a bunch of them. We don't want 5 links like this in the footer, but I don't think we want them in site content either?

A sitemap could also solve this problem, but I think we mostly just want them linked in some discreet fashion.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Improvement Minor improvement to code
Projects
None yet
Development

No branches or pull requests

3 participants