Repository navigation
Show the copy button and answer time on touch screens - #390
Conversation
There was a problem hiding this comment.
Copilot was unable to review this pull request because the user who requested the review has insufficient remaining quota.
An organization owner can choose the repository organization as the billing source for reviews requested by members with a paid Copilot license. Learn about Copilot code review billing.
|
All contributors have signed the CLA ✍️ ✅ |
|
I have read the CLA Document and I hereby sign the CLA |
|
Thanks for this. Before we can merge, please add a before/after screenshot or a short screen recording of the change working in the running app (the issue asks for one), plus a line on how you tested it. |
|
Screenshots at 390px with touch emulation ( Before — the copy button and answer time stay hidden: After — the copy button and answer time are visible: How I tested it: I ran the app, enabled touch emulation at 390px, scrolled to a settled reply, and read |
|
Merged, thanks @tianxingjian2406, good catch for phones. |




Fixes #335
What changed
pointer-coarse:opacity-100and always shows on touch screens.Testing
bun run typecheck,bun test components app lib, andbun run lintrun clean (the known ghostty-web WASM baseline failure aside).Need help on this PR? Tag
@codesmith-botwith what you need.