Modal: Add support for .modal-dialog-scrollable #61

Closed
opened 2020-01-21 22:00:47 +08:00 by modmatrix · 3 comments
modmatrix commented 2020-01-21 22:00:47 +08:00 (Migrated from github.com)

It'd be nice (and pretty easy) to add support for the .modal-dialog-scrollable class:

https://getbootstrap.com/docs/4.4/components/modal/#scrolling-long-content

Right now, it isn't possible to use this feature without resorting to hacks.

(I might submit a pull request for this when I have time – this issue is just a reminder for myself, or someone else, to do that.)

It'd be nice (and pretty easy) to add support for the .modal-dialog-scrollable class: https://getbootstrap.com/docs/4.4/components/modal/#scrolling-long-content Right now, it isn't possible to use this feature without resorting to hacks. (I might submit a pull request for this when I have time – this issue is just a reminder for myself, or someone else, to do that.)
videk1 commented 2020-06-15 15:47:48 +08:00 (Migrated from github.com)

Please add support for this. Much needed!

Please add support for this. Much needed!
Morgul commented 2020-06-26 03:05:49 +08:00 (Migrated from github.com)

I would be fine if someone wants to add that. I'll gladly review the code and merge it in. I, however, am doing active development on this project, so this will have to come from the community.

I would be fine if someone wants to add that. I'll gladly review the code and merge it in. I, however, am doing active development on this project, so this will have to come from the community.
doublesharp commented 2021-04-30 01:25:05 +08:00 (Migrated from github.com)

@Morgul - this tripped me up because in the 3.0.7 version the dist/ui-bootstrap.js version is still at 3.0.6. The 3.0.7 ones have the version in the filename, but it was unexpected for them to not match, so by including the file without the version in the name this functionality was missing.

@Morgul - this tripped me up because in the 3.0.7 version the dist/ui-bootstrap.js version is still at 3.0.6. The 3.0.7 ones have the version in the filename, but it was unexpected for them to not match, so by including the file without the version in the name this functionality was missing.
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: invisi/ui-bootstrap4#61