idux icon indicating copy to clipboard operation
idux copied to clipboard

[WIP]fix(comp:notification): include `useZIndex` to notification wrapper

Open sallerli1 opened this issue 1 year ago • 2 comments

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

sallerli1 avatar Aug 08 '23 13:08 sallerli1

This preview will be available after the AzureCI is passed.

idux-bot[bot] avatar Aug 08 '23 13:08 idux-bot[bot]

Codecov Report

Merging #1644 (e70156d) into main (97bdafb) will increase coverage by 0.00%. The diff coverage is 100.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:

codecov[bot] avatar Aug 08 '23 13:08 codecov[bot]