fix: reduce website footer #4546
Conversation
Ok, here's my pass at reducing the vertical height of the footer. Long term I think we can and should do more, this is just a quick initial cleanup pass: - Left: removed three links (so both columns are down to 5). These are a judgement call, e.g. I felt like 'Docker migration' is a bit of marketing we'd want to leave for now. - Right: remove Blog (it's already at the top of the page). - Right: remove planning and roadmap, both not really useful link and accessible from the Github link above. - Put copyright and license on one line. Overall, this reduces the footer height from 11 lines of text to 7. Signed-off-by: Tim deBoer <git@tdeboer.ca>
deboer-tim
requested review from
slemeur,
themr0c,
benoitf and
a team
as code owners
deboer-tim
requested review from
jeffmaury and
lstocchi
and removed request for
a team
Keep the discord one and replace the others chat links by a link to https://github.com/containers/podman-desktop#communication |
|
@benoitf I didn't think the (previous link) was very useful. How about pointing them to the planning board or current sprint instead? @jeffmaury Makes sense, will do. |
|
@deboer-tim yes as you suggested, we need a redirect link, current link is not good 👍 (I would prefer 'current sprint' link as default to backlog is not really great as well (you need to click on a link again) I agree that footer links are like direct calls and as Jeff pointed out, when there are several links, just take one |
Reduced four chat links to just Discord and a link to Communication section in the readme. Added back in a link to the sprint planning board. Removed Compose from the doc links to even things out again. Signed-off-by: Tim deBoer <git@tdeboer.ca>
|
Pushed another commit:
|
|
Good simplification! |
What does this PR do?
Ok, here's my pass at reducing the vertical height of the footer. Long term I think we can and should do more, this is just a quick initial cleanup pass:
Overall, this reduces the footer height from 11 lines of text to 7.
Screenshot/screencast of this PR
Before:
After (updated to latest commit):
What issues does this PR fix or reference?
Fixes #4299.
How to test this PR?
yarn website:dev