-
-
Notifications
You must be signed in to change notification settings - Fork 184
New issue
Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? # to your account
close button inside modal? #13
Comments
Hi, I'm thinking about this possibility but I would like to keep the close button always visible at the same place, even if the modal is bigger than the screen. I also plan to run some UX tests about this configuration, and will update tingle consequently. |
Currently I'm doing it in onOpen callback, but of course there's a bit of a delay, it seems onOpen is called only after animation finishes.
|
Indeed, onOpen is called at the end of the transition. I will consider adding an option to choose the close button location. |
OK, either an option, or an onBeforeOpen method or something :-) |
I think this improvement would be great! In my case the usability will be better |
I'll review the structure for 1.0 release and try to implement this feature as well. |
+1 I've got around this by adding a close button to the footer, and positioning the footer at the top of the modal. |
Hello! I have a quick suggestion! I think it would be super helpful on the documentation to label the modalSurprise button with something other than a "?". I was STRUGGLING to figure out how to get my youtube video to stop playing/close on exit and had no idea the solution was there. Maybe label it Iframe modal etc. Either way HUGE THANKS for this. |
I have a CSS solution for not moving the close button but instead makes the existing close button 'sticky' and then you can adjust the position to your liking.
then on mobile i used this CSS, but is optional depending upon the margins etc specified above. |
I'd like to put the close button inside the modal window box, that way I can position it with CSS relative to the box.
The text was updated successfully, but these errors were encountered: