Skip to content

Commit

Permalink
fix(modal): fix top border radius when allowMultiple is true
Browse files Browse the repository at this point in the history
When a modal had `allowMultiple:true` set, the top border radius was not set anymore.
This regression happens since an additional dimmer was introduced in #119 to prevent the ability to interact with modals that are not the focused one.
This PR now takes care of a possible existing extra dimmer and shows top border radius correctly again

Closes #308
  • Loading branch information
lubber-de authored and Sean committed Dec 17, 2018
1 parent 9a80898 commit 574830e
Showing 1 changed file with 4 additions and 2 deletions.
6 changes: 4 additions & 2 deletions src/definitions/modules/modal.less
Original file line number Diff line number Diff line change
Expand Up @@ -40,8 +40,10 @@
will-change: top, left, margin, transform, opacity;
}

.ui.modal > :first-child:not(.icon),
.ui.modal > .icon:first-child + * {
.ui.modal > :first-child:not(.icon):not(.dimmer),
.ui.modal > .icon:first-child + *,
.ui.modal > .dimmer:first-child + *:not(.icon),
.ui.modal > .dimmer:first-child + .icon + * {
border-top-left-radius: @borderRadius;
border-top-right-radius: @borderRadius;
}
Expand Down

0 comments on commit 574830e

Please sign in to comment.