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

fix: change Ok to OK #32259

Merged
merged 4 commits into from
Sep 23, 2021
Merged

fix: change Ok to OK #32259

merged 4 commits into from
Sep 23, 2021

Conversation

chj-damon
Copy link
Contributor

@chj-damon chj-damon commented Sep 23, 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

close #32250

💡 Background and solution

the word Ok is not a formal abbreviations, it needs to be changed to OK.

📝 Changelog

Language Changelog
🇺🇸 English 🐞 fix Ok abbreviation to be OK
🇨🇳 Chinese 🐞 修复 英文中Ok缩写为OK

☑️ 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 Sep 23, 2021

@@ -11,7 +11,7 @@ As Alan Cooper states:「Where there is output, let there be input」. This is

## In-Page Editing

<img class="preview-img" align="right" alt="Example of click to edit" description="Status 1: Common browsing mode, do not distinguish between editable and non-editable lines;<br>Status 2: On mouse hover, the background is backlit with yellow. A tool tip invites the user to 'Click to edit';<br>Status 3: Once the user clicks on the title, the form elements 「Input box」, 「Ok」 and 「Cancel」 appear, and the cursor is positioned in the 「input box」." src="https://gw.alipayobjects.com/zos/rmsportal/EXKwsvUkIUNkHBSsOlRi.png">
<img class="preview-img" align="right" alt="Example of click to edit" description="Status 1: Common browsing mode, do not distinguish between editable and non-editable lines;<br>Status 2: On mouse hover, the background is backlit with yellow. A tool tip invites the user to 'Click to edit';<br>Status 3: Once the user clicks on the title, the form elements 「Input box」, 「OK」 and 「Cancel」 appear, and the cursor is positioned in the 「input box」." src="https://gw.alipayobjects.com/zos/rmsportal/EXKwsvUkIUNkHBSsOlRi.png">
Copy link
Member

Choose a reason for hiding this comment

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

CI 挂了,你这个不会是 项目搜索 所有的 Ok 进行替换吧。。。。

Copy link
Contributor Author

Choose a reason for hiding this comment

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

我只是修改了文案,怎么会导致test都跑挂呢

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.

重新跑了一下yarn test:update,更新了snapshot

Copy link
Member

Choose a reason for hiding this comment

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

其实那几个 Demo ,可以在 md 上 改成 OK ,统一一些

@codecov
Copy link

codecov bot commented Sep 23, 2021

Codecov Report

Merging #32259 (b0c05ad) into master (5445fba) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##            master    #32259   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files          414       414           
  Lines         7571      7571           
  Branches      2125      2125           
=========================================
  Hits          7571      7571           
Impacted Files Coverage Δ
components/date-picker/locale/pt_PT.tsx 100.00% <ø> (ø)
components/date-picker/locale/sl_SI.tsx 100.00% <ø> (ø)
components/locale/cs_CZ.tsx 100.00% <ø> (ø)
components/locale/kk_KZ.tsx 100.00% <ø> (ø)
components/locale/ms_MY.tsx 100.00% <ø> (ø)

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 5445fba...b0c05ad. Read the comment docs.

@vnues
Copy link

vnues commented Sep 23, 2021

哈哈哈 本来我还想修来着,老哥够快速的

@afc163 afc163 merged commit e7f5030 into ant-design:master Sep 23, 2021
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.

日期控件里面的确认按钮"Ok"改成"OK"
4 participants