material-ui icon indicating copy to clipboard operation
material-ui copied to clipboard

[ChipDelete][joy] Add onDelete prop to ChipDelete

Open sai6855 opened this issue 3 years ago • 5 comments

Closes: https://github.com/mui/material-ui/issues/35188

This PR adds onDelete prop to ChipDelete component, onDelete fires when user clicks on component or presses Backspace or enter.

sai6855 avatar Dec 09 '22 15:12 sai6855

Messages
:book: Netlify deploy preview: https://deploy-preview-35412--material-ui.netlify.app/

@mui/joy: parsed: +0.07% , gzip: +0.14%

Details of bundle changes

Generated by :no_entry_sign: dangerJS against 0e7dc3b36aacd86e00802e87a8fb3091989c5dc7

mui-bot avatar Dec 09 '22 15:12 mui-bot

@ZeeshanTamboli can you review this PR

sai6855 avatar Dec 13 '22 07:12 sai6855

@ZeeshanTamboli updated code as described, but ci/circleci: checkout test is failing. i'm not able to fix it, can you check it once

sai6855 avatar Dec 13 '22 16:12 sai6855

@ZeeshanTamboli updated changes also added onDelete prop description in docs. preview: https://deploy-preview-35412--material-ui.netlify.app/joy-ui/react-chip/#delete-button

but test-types test is failing not sure why, it is asking to run yarn docs:typescript:formatted but when i ran yarn docs:typescript:formatted DeletableChip.tsx.preview file is got deleted.

sai6855 avatar Dec 14 '22 06:12 sai6855

but test-types test is failing not sure why, it is asking to run yarn docs:typescript:formatted but when i ran yarn docs:typescript:formatted DeletableChip.tsx.preview file is got deleted.

Then I would suggest to remove onDelete code from the demo. Just the text is enough.

ZeeshanTamboli avatar Dec 14 '22 13:12 ZeeshanTamboli