Always show chart axis titles on touchscreens

ForrestWeiswolf commented 1 month ago

This should

Preserve current behavior (axis labels hidden by default, visible on hover, toggleable via ? icon) on non-touchscreen devices
Show the axis labels by default on touchscreen devices, since many of them don't have a hover behaviour at all, while still allowing it to be toggled with the ? icon

AFAIK there's no fully-reliable way to detect whether a device has mouse/touchscreen but this will work in most cases, and if it uses the current behaviour on some uncommon touchscreen devices (e.g. some models of Blackberry?) or the touchscreen behaviour on some laptops-with-touchscreen, that's basically okay.

vercel[bot] commented 1 month ago

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

Name	Status	Preview	Comments	Updated (UTC)
forecast-bot	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Aug 15, 2024 3:51pm

adam-binks commented 1 month ago

Lgtm! (Haven't tested)

ForrestWeiswolf commented 1 month ago

Tested on desktop (screen recording below) + tested the preview branch on my android phone (no screen recording because I don't know how to record video of the screen on it). Can't log in on preview branches, but a user's public calibration graph can be seen while logged out and nothing about the graph axes differed when logged in.

https://github.com/user-attachments/assets/78285fcb-e3a1-44ee-93df-42b48a625ba4

Sage-Future / fatebook

Always show chart axis titles on touchscreens #32