You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Describe the bug
When opening the "Display options" popover and setting "Row height" to "Custom", users are unable to clear out the "Lines per row" input value to type in another. Instead they must either use the slider, spinner, or highlight the current value and type over it.
Impact and severity
Low impact and severity. There are easy ways to work around it, and it's just an annoying UX paper cut.
Environment and versions
EUI version: v90.0.0
React version: 17.0.2
Kibana version (if applicable): 8.12
Browser: Chrome
Operating System: macOS
To Reproduce
Steps to reproduce the behavior:
Go to any EUI data grid (e.g. Discover in Kibana).
Click on the "Display options" button.
Set "Row height" to "Custom".
Attempt to clear the "Lines per row" input value.
Observe that the value cannot be cleared.
Expected behavior
Allow the "Lines per row" input to be cleared out, then highlight the input as invalid, and if no valid value is entered, reset to the previous value upon dismissing the popover.
Describe the bug
When opening the "Display options" popover and setting "Row height" to "Custom", users are unable to clear out the "Lines per row" input value to type in another. Instead they must either use the slider, spinner, or highlight the current value and type over it.
Impact and severity
Low impact and severity. There are easy ways to work around it, and it's just an annoying UX paper cut.
Environment and versions
To Reproduce
Steps to reproduce the behavior:
Expected behavior
Allow the "Lines per row" input to be cleared out, then highlight the input as invalid, and if no valid value is entered, reset to the previous value upon dismissing the popover.
Minimum reproducible sandbox
Reproducible in any EuiDataGrid. Used the "Core concepts" example for the sandbox: https://codesandbox.io/s/5kw62m?file=/demo.js
Screenshots
lines_per_row.mp4