ciena-frost / ember-frost-core

http://ciena-frost.github.io/ember-frost-core/
MIT License
18 stars 42 forks source link

Move notification z-index above modal z-index #458

Closed EWhite613 closed 7 years ago

EWhite613 commented 7 years ago

This project uses semver, please check the scope of this pr:

CHANGELOG

EWhite613 commented 7 years ago

Reference: https://github.com/ciena-frost/ember-frost-notifier/pull/41

sglanzer-deprecated commented 7 years ago

Makes sense to me - I'll just confirm this with UX

sglanzer-deprecated commented 7 years ago

Approved

Approved with PullApprove

sglanzer-deprecated commented 7 years ago

Approved - Adjusted navigation to just below app bar per UX

Approved with PullApprove

coveralls commented 7 years ago

Coverage Status

Changes Unknown when pulling 45a0b9ae4c4ac95843a061673b17a25cb93f2c0d on EWhite613:z-index-changes into on ciena-frost:master.

coveralls commented 7 years ago

Coverage Status

Changes Unknown when pulling 45a0b9ae4c4ac95843a061673b17a25cb93f2c0d on EWhite613:z-index-changes into on ciena-frost:master.