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 responsive footer element #1122

Closed
wants to merge 2 commits into from
Closed

Add responsive footer element #1122

wants to merge 2 commits into from

Conversation

tfc
Copy link
Contributor

@tfc tfc commented Sep 16, 2023

As discussed in the meeting, this is my attempt to mimic the original footer of nixos.org, but described with tailwind CSS.

ref #1125

---

<footer>
Footer
<style>
.nixlogo {
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

not sure what's the best way to do this. The good part about this is that it is defined in the same file as it's used. but if we need this svg somewhere else too, we would need a more central place.
Of course, it could also just be an tag that links to an svg file... not sure why it is like this in the original nixos.org implementation?

Copy link
Member

@garbas garbas Sep 19, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think we should create a custom icon set as described in astro-icon.

@thilobillerbeck
Copy link
Collaborator

looks good 👍

since this is already in main, there is no need to merge imo

@garbas
Copy link
Member

garbas commented Sep 19, 2023

@tfc awesome to have so quick turn around. I've protected the main branch going forward so we go via review process. I'll close this PR since it is already in main.

@garbas garbas closed this Sep 19, 2023
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.

3 participants