Wrong selection animation about vaadin-grid

[amahdy]

Root cause
In Vaadin Grid, by default v-grid-row-focused class is added to the first tr. It seems that in the current design, one exact tr must have this class.

This conflicts with the current CSS

:focus .v-grid-row-focused {
...
}

Since the class v-grid-row-focused is always present, extra unneeded animation happens.

Proposed solutions

1. Update our CSS rules and probably not use :focus.
2. If it's a bug against Vaadin Grid design, remove the class v-grid-row-focused when it's not needed.

_Note_ v-grid-row-focused is completely different than v-grid-row-selected.

[jouni]

I’ve noticed this also, but haven’t considered it a priority yet. If possible, I would definitely opt for removing the v-grid-row-focused classname from all the rows when the grid if blurred/unfocused.

If we do that, we can also stop using :focus at the same time.

[amahdy]

Working on a fix now for the main Vaadin Grid code. This should fix many issues in v-grid.

[amahdy]

https://dev.vaadin.com/ticket/18521

[jouni]

Thanks to @Artur-, this issue is progressing 👍

[jouni]

Probably not worth fixing any more. Closing as “won’t fix”.