Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

feat: add rootClassName #445

Merged
merged 1 commit into from
Apr 8, 2022

Conversation

heiyu4585
Copy link
Contributor

[中文版模板 / 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

ant-design/ant-design#34318

💡 Background and solution

📝 Changelog

Language Changelog
🇺🇸 English add rootClassName
🇨🇳 Chinese 添加 rootClassName

☑️ 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

@heiyu4585 heiyu4585 closed this Apr 5, 2022
@heiyu4585 heiyu4585 force-pushed the fix-rootClassName branch from aa8e960 to 1b5cbab Compare April 5, 2022 13:52
@heiyu4585 heiyu4585 reopened this Apr 5, 2022
@heiyu4585 heiyu4585 force-pushed the fix-rootClassName branch from 4ceb8f6 to a670a00 Compare April 5, 2022 14:08
src/Menu.tsx Outdated
*
* - OpenTransitionName
* - OpenAnimation
* - OnDestroy
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

备注里的是方法,不用转换大小写

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

prettier自动转化的,也不转么

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ready

style="opacity: 0; pointer-events: none;"
>
<ul
class="rc-menu rc-menu-sub rc-menu-vertical custom-className"
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

rootClassName 总是出现在最外层 dom 上,这边出现两个是不对的。应该就在最外面的 div 上面。

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ready

@codecov
Copy link

codecov bot commented Apr 6, 2022

Codecov Report

Merging #445 (7f3dfcd) into master (1b5cbab) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #445   +/-   ##
=======================================
  Coverage   99.85%   99.85%           
=======================================
  Files          25       25           
  Lines         674      674           
  Branches      175      175           
=======================================
  Hits          673      673           
  Misses          1        1           
Impacted Files Coverage Δ
src/Menu.tsx 100.00% <ø> (ø)
src/SubMenu/PopupTrigger.tsx 100.00% <ø> (ø)
src/context/MenuContext.tsx 100.00% <ø> (ø)

📣 Codecov can now indicate which changes are the most critical in Pull Requests. Learn more

@heiyu4585 heiyu4585 reopened this Apr 8, 2022
@zombieJ zombieJ merged commit 6cd311c into react-component:master Apr 8, 2022
@zombieJ
Copy link
Member

zombieJ commented Apr 8, 2022

@MadCcc pls help to release related minor version.

@MadCcc
Copy link
Member

MadCcc commented Apr 8, 2022

v9.5.4 published

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants