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
Tune the behavior of the Paratime Picker - Clicky version #468
Conversation
Deployed to Cloudflare Pages
|
1ab5efa
to
af259f4
Compare
After discussing this with @donouwens , we came to the conclusion that this is not (necessarily) what we want to do here. |
df5239f
to
633ad7f
Compare
633ad7f
to
fc5c60b
Compare
Updated as discussed |
1966143
to
fa3ead9
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yay
setHoveredLayer(layer) | ||
}} | ||
onClick={() => setSelectedLayer(layer)} | ||
onMouseLeave={() => { | ||
setHoveredLayer() |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We can later simplify to css :hover
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Agreed.
fa3ead9
to
45b4555
Compare
45b4555
to
aa6a96b
Compare
Rationale coming at the next chance to talk.
See #479 for a no-click alternative.