Improve z-index for logs

alertifyjs / alertify.js

JavaScript Alert/Notification System

https://alertifyjs.org

MIT License

521 stars 104 forks source link

Improve z-index for logs #88

Open quedicesebas opened 8 years ago

quedicesebas commented 8 years ago

I'm using this library with Ionic (http://ionicframework.com/) and now is not working because you put the z-index in 1, and that causes alerts be behind views (with z-index 3)

ollie314 commented 8 years ago

Hi, Thanks for the report. Do you have a snippet to send to us so that we have to reproduce the issue with our specific case ? Regards

quedicesebas commented 8 years ago

I'm a little busy now but as soon as I can I'm gonna make a codepen for you

chrisvidal commented 8 years ago

same issue here,

just need to adjust the .alert-logs CSS class z-index per default to a high value instead of 1 in the alertify.css
add your own stylesheet with just this .alert-logs with a higher z-index

bradberger commented 8 years ago

@chrisvidal Thanks for the tip. That's definitely a good solution.

I think it would make sense to adjust the z-index in the next patch release, so I'll keep this open so we won't forget about it. But in the meantime, the suggested approach should definitely work.

jor3l commented 8 years ago

I just added the pull request