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

feat: add youtube link and create footer nav #476

Merged
merged 3 commits into from Feb 12, 2024
Merged

feat: add youtube link and create footer nav #476

merged 3 commits into from Feb 12, 2024

Conversation

dni
Copy link
Member

@dni dni commented Feb 12, 2024

closes #445

add youtube link
and add footer nav

screenshot-1707725706

@dni dni requested a review from michael1011 February 12, 2024 08:17
Copy link

vercel bot commented Feb 12, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated (UTC)
webapp ✅ Ready (Inspect) Visit Preview Feb 12, 2024 9:45pm
webapp-mainnet ✅ Ready (Inspect) Visit Preview Feb 12, 2024 9:45pm

Comment on lines 65 to 69
<Show when={statusUrl}>
<a class="external" target="_blank" href={statusUrl}>
{t("status")}
</a>
</Show>
Copy link
Member

Choose a reason for hiding this comment

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

Nack
How often do you click a status link? Almost never. Makes no sense to show it to users all the time

Copy link
Member

Choose a reason for hiding this comment

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

agree

Copy link
Member

Choose a reason for hiding this comment

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

plus the web app will scream in your face if sth is not working anyways.

@kilrau
Copy link
Member

kilrau commented Feb 12, 2024

  • I'd actually move the blog into footer too. The header menu is too much and the ame logic applies: blog is just not crucial functionality
  • I didn't find the branding link

closes #445

add youtube link and remove blog link from footer
move status link from footer to main nav
@dni dni changed the title feat: add youtube link and move status feat: add youtube link and create footer nav Feb 12, 2024
@dni dni requested a review from michael1011 February 12, 2024 21:15
@michael1011 michael1011 merged commit 9be3c87 into main Feb 12, 2024
4 checks passed
@michael1011 michael1011 deleted the links branch February 12, 2024 21:47
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.

Add Branding, Youtube Link
3 participants