Skip to content
This repository has been archived by the owner on May 4, 2018. It is now read-only.

Valo date-picker: date cell hover → active #5

Closed
jouni opened this issue Sep 8, 2017 · 1 comment
Closed

Valo date-picker: date cell hover → active #5

jouni opened this issue Sep 8, 2017 · 1 comment

Comments

@jouni
Copy link
Member

jouni commented Sep 8, 2017

From @jouni on August 25, 2017 12:55

The selection box should only appear on :active, not on :hover, with the exception of touch devices, where we should not show the :active style when scrolling. That’s not really possible (no [scrolling] state available), so just avoid showing the :active state altogether for touch devices.

Basically, it should work like in the Material theme, you can probably copy-paste it from there: https://github.com/vaadin/vaadin-themes/blob/master/material/vaadin-date-picker.html#L75-L108

Copied from original issue: vaadin/vaadin-themes#44

@jouni
Copy link
Member Author

jouni commented Sep 12, 2017

Fixed this issue (although not how I described above) in 7e67dcf

@jouni jouni closed this as completed Sep 12, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant