idux
idux copied to clipboard
[WIP]fix(comp:notification): include `useZIndex` to notification wrapper
PR Checklist
Please check if your PR fulfills the following requirements:
- [x] The commit message follows our guidelines
- [x] Tests for the changes have been added/updated or not needed
- [x] Docs and demo have been added/updated or not needed
What is the current behavior?
notificaiont wrapper的z-index是写死的,导致如果配置了overlayZIndex,可能会出现message一直在底部的问题
What is the new behavior?
使用 useZIndex 维护 wrapper 的 zIndex
Other information
This preview will be available after the AzureCI is passed.
Codecov Report
Merging #1644 (e70156d) into main (97bdafb) will increase coverage by
0.00%
. The diff coverage is100.00%
.
:exclamation: Current head e70156d differs from pull request most recent head 75e1813. Consider uploading reports for the commit 75e1813 to get more accurate results
@@ Coverage Diff @@
## main #1644 +/- ##
=======================================
Coverage 92.62% 92.63%
=======================================
Files 345 345
Lines 32199 32222 +23
Branches 3677 3678 +1
=======================================
+ Hits 29825 29848 +23
Misses 2374 2374
Files Changed | Coverage Δ | |
---|---|---|
...mponents/notification/src/NotificationProvider.tsx | 98.47% <100.00%> (+0.14%) |
:arrow_up: |