Feat: move remove filter button and display it on hover

[thibault-barrat]

• Display the close button only when hovering the input
• Move the close button to the top right of the input

Enregistrement.de.l.ecran.2022-12-01.a.15.08.39.mov

[WiXSL]

I really like it! 👍

[slax57]

IMO this should be an opt-in feature, because it is a regression in terms of accessibility.

Besides, if it's a new feature, it should target the next branch.

[fzaninotto]

I like it!

Why do you see it as a regression in accessibility?

In my opinion, this feature needs a bit of visual polish on the background and position of the close button, the hover zone, and the tooltip.

[slax57]

I like it!

Why do you see it as a regression in accessibility?

In my opinion, this feature needs a bit of visual polish on the background and position of the close button, the hover zone, and the tooltip.

Because you can no longer use the Clear button with keyboard only

[antoinefricker]

I make a reservation on 3 points:

• filter UI do not display its clearable state without hovering it; I would prefer the button to be displayed by default
• chips are now clickable, without consequent action; they have the UI of a toggler (enabled/disabled) but without being one
• dark theme should also be evaluated

[fzaninotto]

After thinking about the accessibility regression, we won't move forward with this change. We'll keep the less visually appealing "always on" clear button to remain accessible.

thanks for your contribution anyway!

[fzaninotto]

Superseded by #9224