search

angular-ui / bootstrap

PLEASE READ THE PROJECT STATUS BELOW. Native AngularJS (Angular) directives for Bootstrap. Smaller footprint (20kB gzipped), no 3rd party JS dependencies (jQuery, bootstrap JS) required. Please read the README.md file before submitting an issue!
http://angular-ui.github.io/bootstrap/
MIT License
14.29k stars 6.73k forks source link

Modal less configurable since replace: true removed #6471

Open connorwyatt opened 7 years ago

connorwyatt commented 7 years ago

Bug description:

The $uibModalStack directive uses hard-coded values for z-index and animation class, meaning that they are not configurable. Before the removal of replace: true it was possible to do so as your windowTemplate was used in place of the default wrapper.

I am asking for a configuration option to allow both to be specified.

I am aware that the z-index needs to include some way of setting higher indexed modals above lower indexed ones so I am suggesting having a baseZIndex property in the config.

Config

Link to minimally-working plunker that reproduces the issue:

N/A

Steps to reproduce the issue:

N/A

Version of Angular, UIBS, and Bootstrap

Angular: 1.5.8

UIBS: 2.5.0

Bootstrap: N/A

loxy commented 7 years ago

I can confirm this. Since: https://github.com/angular-ui/bootstrap/commit/96d52ce150219c4071ef4ebf7c4ec0f7ab3bdb81 it is not possible anymore to specify the z-index by editing the template, because this part has been removed.

But we NEED such a possibility! Please open https://github.com/angular-ui/bootstrap/issues/3580 and https://github.com/angular-ui/bootstrap/issues/2373 again!

loxy commented 7 years ago

Some words to our usecase. We are running a portal app which includes several apps in an iframe, always one at a time. We can open a modal in an app or in the portal. And so we control over the z-index especially the place the backdrop right...