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

Improve site footer #26

Merged
merged 5 commits into from
Nov 12, 2023
Merged

Improve site footer #26

merged 5 commits into from
Nov 12, 2023

Conversation

per1234
Copy link
Owner

@per1234 per1234 commented Nov 12, 2023

No description provided.

The standard style for links established by the hugo-PaperMod theme used by the website is for the text to be
underlined.

That style was somehow lost when a custom footer was implemented for the site. It is hereby restored. The approach used
to obtain the underline comes from the theme's CSS code:

https://github.com/adityatelange/hugo-PaperMod/blob/v7.0/assets/css/common/footer.css#L26
I think it looks better with a margin on each side.
Previously a crude approach was taken to graphically separating the footer components.

A superior approach is to use borders. This gives more control over the appearance which allows achieving an
aesthetically superior design.
This will make it easier for visitors who are interested in the source code of the project to find the GitHub
repository.
The default footer height set by the hugo-PaperMod theme is far larger than necessary, which is unsightly in addition to
wasting page space.
@per1234 per1234 added type: enhancement Proposed improvement topic: code Related to content of the project itself labels Nov 12, 2023
@per1234 per1234 self-assigned this Nov 12, 2023
@per1234 per1234 merged commit e32d2b6 into main Nov 12, 2023
44 of 46 checks passed
@per1234 per1234 deleted the footer branch November 12, 2023 08:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
topic: code Related to content of the project itself type: enhancement Proposed improvement
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant