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

Bug: Changelog Entries aren't clickable on mobile #344

Closed
BekahHW opened this issue Aug 26, 2024 · 4 comments · Fixed by #350
Closed

Bug: Changelog Entries aren't clickable on mobile #344

BekahHW opened this issue Aug 26, 2024 · 4 comments · Fixed by #350
Assignees
Labels
🐛 bug Something isn't working 👀 needs triage

Comments

@BekahHW
Copy link
Member

BekahHW commented Aug 26, 2024

Describe the bug

When you're on the changelog page you can't click the title of a post to go to the individual changelog page.

This is happening in Safari and chrome in ios. It's worth noting that clicking the "See more..." isn't the same thing as going to the individual page, as that cuts off the entries.

Steps to reproduce

  1. Go to https://opensauced.pizza/changelog on mobile
  2. Try to click a title. For example:

image
3. Nothing happens

@BekahHW BekahHW added 🐛 bug Something isn't working 👀 needs triage labels Aug 26, 2024
@nickytonline
Copy link
Member

It looks like on mobile the titles get rendered as text and not links.

@dev-phantom
Copy link
Contributor

dev-phantom commented Sep 7, 2024

yea
adding this
<Link href={/changelog/${slug?.current}} className="hover:text-brandOrange hover:decoration-brandOrange transition-all" > {title} </Link>

here


fixes the issue

@dev-phantom
Copy link
Contributor

.take

pushing this immediatly my PR gets merged

Copy link
Contributor

github-actions bot commented Sep 7, 2024

Thanks for taking this on! If you have not already, join the conversation in our Discord

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🐛 bug Something isn't working 👀 needs triage
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants