Refactor DirectoryItemsInput to use MUI Select with improved UX #949
Conversation
- Replace input by MUI Select/OutlinedInput component - Add proper label support with InputLabel - Implement chip-based multi-selection with click handlers - Add event propagation control for chip interactions - Update OverflowableChip and OverflowableChipWithHelperText with boxSx prop - Prevent tooltip click propagation in OverflowableText - Update demo with DirectoryItemsInput example Signed-off-by: Florent MILLOT <[email protected]>
| } | ||
| {...otherProps} | ||
| /> | ||
| <Box padding="3px" sx={{ cursor: 'default', ...boxSx }} onClick={(e) => e.stopPropagation()}> |
There was a problem hiding this comment.
If I understood well the point of all this is to have the unclickable aura around all the chips ?
There was a problem hiding this comment.
Should I comment this part?
There was a problem hiding this comment.
No, the code is clear.
This is the intent that is a bit strange for me (3 pixels unclickable around a clickable Chip inside a clickable Select...)
But I can see that this is in the jira ticket.
This comment was marked as resolved.
This comment was marked as resolved.
Sorry, something went wrong.
Co-authored-by: Mathieu Deharbe <[email protected]>
I will try to fix the first comment but for the second one I think it's original behavior |
First issue fixed here http://github.com/gridsuite/gridstudy-app/pull/3542 |
|
Uh oh!
There was an error while loading. Please reload this page.