Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

fix: Add missing tooltip-zIndex #30432

Merged
merged 2 commits into from
May 7, 2021
Merged

fix: Add missing tooltip-zIndex #30432

merged 2 commits into from
May 7, 2021

Conversation

zombieJ
Copy link
Member

@zombieJ zombieJ commented May 7, 2021

[中文版模板 / Chinese template]

🤔 This is a ...

  • New feature
  • Bug fix
  • Site / documentation update
  • Demo update
  • Component style update
  • TypeScript definition update
  • Bundle size optimization
  • Performance optimization
  • Enhancement feature
  • Internationalization
  • Refactoring
  • Code style optimization
  • Test Case
  • Branch merge
  • Other (about what?)

🔗 Related issue link

💡 Background and solution

📝 Changelog

Language Changelog
🇺🇸 English Export Tooltip zIndex prop typescript definition.
🇨🇳 Chinese Tooltip 导出 zIndex typescript 定义。

☑️ Self Check before Merge

⚠️ Please check all items below before review. ⚠️

  • Doc is updated/provided or not needed
  • Demo is updated/provided or not needed
  • TypeScript definition is updated/provided or not needed
  • Changelog is provided or not needed

@github-actions
Copy link
Contributor

github-actions bot commented May 7, 2021

@codecov
Copy link

codecov bot commented May 7, 2021

Codecov Report

Merging #30432 (38e6029) into master (8e76c1c) will not change coverage.
The diff coverage is n/a.

❗ Current head 38e6029 differs from pull request most recent head b660a25. Consider uploading reports for the commit b660a25 to get more accurate results
Impacted file tree graph

@@            Coverage Diff            @@
##            master    #30432   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files          397       397           
  Lines         7514      7514           
  Branches      2046      2046           
=========================================
  Hits          7514      7514           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 8e76c1c...b660a25. Read the comment docs.

@github-actions
Copy link
Contributor

github-actions bot commented May 7, 2021

Size Change: 0 B

Total Size: 831 kB

ℹ️ View Unchanged
Filename Size Change
./dist/antd-with-locales.min.js 340 kB 0 B
./dist/antd.compact.min.css 63.9 kB 0 B
./dist/antd.dark.min.css 65.2 kB 0 B
./dist/antd.min.css 63.9 kB 0 B
./dist/antd.min.js 298 kB 0 B

compressed-size-action

@afc163
Copy link
Member

afc163 commented May 7, 2021

新特性还是修复?

@zombieJ
Copy link
Member Author

zombieJ commented May 7, 2021

代码里 Tooltip 属性是透传给 Trigger 的,原本就支持。但是没有 TS 所以云凤蝶解析不到,补了一下 TS。

@afc163 afc163 merged commit 11b7676 into master May 7, 2021
@afc163 afc163 deleted the tooltip-zIndex branch May 7, 2021 09:38
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants