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

add option to add links to the footer of the page #168

Merged
merged 1 commit into from
Feb 21, 2024

Conversation

dholbach
Copy link
Contributor

grafik

The header menu might get a bit busy for pages with a lot of sections and links. So I thought it would be nice to add e.g. less important links to sub-pages you might be legally required to have, like license, imprint, copyright, etc. to the footer of the page.

@dholbach dholbach changed the title add an option to add links at the bottom of the page as well add option to add links to the footer of the page Feb 17, 2024
@zjedi
Copy link
Owner

zjedi commented Feb 18, 2024

@dholbach Thanks. I can see the use case, but I am not convinced about solution. The footer looks quite thick.

Why not adding the links directly to existing footer section?

"Additional Links" section is quite obvious, could use get just the links without the title?

If you really want to have it in two separate rows, can we get rid of the the top hr?

Signed-off-by: Daniel Holbach <daniel.holbach@gmail.com>
@dholbach
Copy link
Contributor Author

Thanks for your feedback. Is this more to your liking now?

grafik

@zjedi zjedi merged commit 2d94f74 into zjedi:master Feb 21, 2024
6 checks passed
@dholbach dholbach deleted the add-bottom-links branch February 21, 2024 22:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants