Skip to content

Commit

Permalink
Merge pull request #1675 from cozy/bug-fix
Browse files Browse the repository at this point in the history
  • Loading branch information
ptbrowne authored Nov 27, 2020
2 parents c2aa34e + 98cbe38 commit 8838068
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion react/CozyDialogs/Readme.md
Original file line number Diff line number Diff line change
Expand Up @@ -170,7 +170,7 @@ const toggleDialog = dialog => {
</MuiCozyTheme>
</BreakpointsProvider>
{dialogs.map(dialog => (
<button data-test-id={`open-btn-${dialog.name}`} onClick={() => toggleDialog(dialog)}>
<button key={`open-btn-${dialog.name}`} data-test-id={`open-btn-${dialog.name}`} onClick={() => toggleDialog(dialog)}>
Open {dialog.name}
</button>
))}
Expand Down
2 changes: 1 addition & 1 deletion react/CozyDialogs/useCozyDialog.js
Original file line number Diff line number Diff line change
Expand Up @@ -33,7 +33,7 @@ const useCozyDialog = props => {
const dialogProps = {
'aria-labelledby': `modal-title-${id}`,
fullScreen,
open: open || opened,
open: open !== undefined ? open : opened,
onClose,
TransitionComponent,
...otherProps,
Expand Down

0 comments on commit 8838068

Please sign in to comment.