Skip to content

Make theme card tappable on mobile#67

Merged
meninoebom merged 1 commit into
mainfrom
meninoebom/mobile-theme-link
Apr 24, 2026
Merged

Make theme card tappable on mobile#67
meninoebom merged 1 commit into
mainfrom
meninoebom/mobile-theme-link

Conversation

@meninoebom
Copy link
Copy Markdown
Owner

Summary

  • The hover-only link icon was invisible on mobile — no way to reach a theme's permalink page by tapping.
  • Replaced it with a stretched-link overlay: the entire image + body region is now the tap target.
  • Inline markdown links inside body_md stay clickable via a prose-scoped z-index lift.
  • Permalink variant skips the overlay (no pointless self-link).

Test plan

  • Tap theme image on mobile → navigates to /themes/<id>
  • Tap theme body on mobile → navigates to /themes/<id>
  • Tag pills below remain independently clickable
  • Desktop click still works the same

🤖 Generated with Claude Code

The hover-only link icon was invisible on mobile. Replace it with a
stretched-link overlay so the entire image + body region navigates to
the permalink on tap. Markdown links inside body_md stay clickable via
a prose-scoped z-index lift. Permalink variant skips the overlay to
avoid a pointless self-link.

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
@meninoebom meninoebom merged commit 4004e52 into main Apr 24, 2026
2 checks passed
@railway-app railway-app Bot temporarily deployed to Breadcrumbs / breadcrumbs-pr-67 April 24, 2026 07:00 Destroyed
@meninoebom meninoebom deleted the meninoebom/mobile-theme-link branch April 24, 2026 07:00
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.

1 participant