Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Multiple Modals: Closing second modal by clicking dimmer does not work everywhere in dimmer if second modal is smaller than first one #609

Closed
PhilippGrashoff opened this issue Nov 29, 2018 · 10 comments
Assignees

Comments

@PhilippGrashoff
Copy link
Collaborator

Hi there,

if this text isnt explaining well enough, I will add a screencast:

  1. This only occurs if more than one modal is opened and the second modal is smaller than the first modal.
  2. If thats the case, clicking the dimmer to close the second modal does not work in the part of the dimmer that overlays the space of the first modal (second modal is smaller, so dimmer "grows" when opening second one. Inside the "grow" clicking to close does not work).

Suggestion: Fomantic UI has fixed the multiple modal issue too, we run our own fix. Try the fomantic fix and use it if it works well.

@ibelar
Copy link
Contributor

ibelar commented Dec 3, 2018

Submit issue to Fomantic fomantic/Fomantic-UI#284

@PhilippGrashoff
Copy link
Collaborator Author

Hi, my bug is actually quite the contrary: Its about not being able to close the modal by clicking the dimmer. Will make a screencast

@ibelar
Copy link
Contributor

ibelar commented Dec 3, 2018

@PhilippGrashoff - no need for screenshot. The bug is related and cause by onHide callback that is not process correctly when multiple modal. Just add to show it differently 😄

@romaninsh
Copy link
Member

May be related - #436

@romaninsh
Copy link
Member

@PhilippGrashoff
Copy link
Collaborator Author

Hi,

thats not my issue. Seems in this demo all 4 modals are closed at once. I use ATK4 UI 1.6.0, this version does not suffer from all modals being closed at once. Will try to test on develop.

@ibelar
Copy link
Contributor

ibelar commented Dec 4, 2018

@PhilippGrashoff - I think I see what you mean. When saying that dimmer grow do you mean that dimmer should take up the space of the second modal open area? As if only the second modal would be use alone? And then clicking that grow region would close second modal only?

This is not current Modal behaviour in Semantic. You can try using the multiple modal demo on their site. https://fomantic-ui.com/modules/modal.html#multiple-modals. When you open the second modal, clicking on the first one in the back (what appear as the growing dimmer area) will not close the second one.

Perhaps this was working before using semantic and the way we implement the fix because positioning of multiple modal was bad. But I think this is the right behaviour now or at least the way it is implement in modal.

Let me know,

@PhilippGrashoff
Copy link
Collaborator Author

Hi,

finally we get together :) Should have made a screencast in the first place. Ok, if thats the implementation in Fomantic-UI, then thats the way it is. Its not a big issue anyway, will post it there and close here.
On the demo page you posted its way more visible as the first modal is dimmed in white and thus differs from the other dimmer. In my page both dimmers are black so there was no difference visible.

@ibelar
Copy link
Contributor

ibelar commented Dec 4, 2018

Good that we understand each other now. But still this allow us to fine a bug in closing multiple modal via dimmer area in semantic. 😄

@GregoryR13
Copy link

Hello,

It works with that :

$('.coupled.modal').modal({ allowMultiple: true, closable: false });

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants