Skip to content

Conversation

@v4rgas
Copy link
Contributor

@v4rgas v4rgas commented Dec 24, 2025

Summary

  • Wrap PostCard and PostHeader in anchor tags for full card clickability
  • Convert internal links (domain, author) to buttons with event propagation prevention
  • Update UpvoteButton onclick handler to accept MouseEvent
  • Clean up unused escapeHtml function from OG image route

- Wrap PostCard and PostHeader in anchor tags for full card clickability
- Convert internal links to buttons with event propagation prevention
- Update UpvoteButton onclick handler to accept MouseEvent
- Remove unused escapeHtml function from OG image route
@emersoftware
Copy link
Owner

gracias amigo por el PR, pero como te comenté por privado el flujo deseado, por diseño, es que el título sea clickeable y abra el link en un tab nuevo para que sea una navegación rápida hacia afuera de the stack

tus cambios obligan que esa navegación sea en dos pasos (click card → post page → click link externo)

si queremos hacer la card más clickeable, podríamos explorar otras alternativas, se me ocurren dos:

  1. en mobile, que la card completa lleve a la pagina externa, fuera de the stack. principalmente porque en mobile no hay hover para el titulo
  2. en desktop, que el hover de la card muestre algun elemento para ir al post page

@v4rgas
Copy link
Contributor Author

v4rgas commented Dec 26, 2025

amigo es fake eso pq aun puedes clickear el nombre de la tarjeta para ir directo al link, solo que la tarjeta de tira al post

@emersoftware
Copy link
Owner

lo fact checkie, pullié tus cambios y al hacer click en el titulo se iba a la pagina del post no al link externo

igual es incomodo que un miss click te lleve al post clickeando la tarjeta en cambio del titulo y viceversa

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.

2 participants