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

Remove recommended title case capitalization tool #5

Merged

Conversation

AlexShan2008
Copy link
Contributor

@AlexShan2008 AlexShan2008 commented Jan 15, 2025

这个 MR 做了什么?

移除了标题大小写转换工具链接及有关描述

为什么提交这个 MR?

  1. 标题大小写转换工具链接已失效
  2. 其它类似工具普遍存在广告和冗余功能

@@ -464,7 +464,7 @@

> Iphone 在短短数年内就挑战了 blackberry 在企业市场中的地位。

1. 文章的标题(title)、出版物名称(包括书籍、音乐和影视作品等)应该使用标题大小写(title case),而非句子大小写(sentence case)。标题大小写可以使用[这个](https://headlinecapitalization.com)工具。
1. 文章的标题(title)、出版物名称(包括书籍、音乐和影视作品等)应该使用标题大小写(title case),而非句子大小写(sentence case)。
Copy link
Member

Choose a reason for hiding this comment

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

没有工具后,可能需要举一些具体的例子?

标题:How to Build a RESTful API with Laravel
电影名称:The Lord of the Rings
书籍标题:A Tale of Two Cities

By the way, 这些例子是正确的吗?

Copy link
Member

Choose a reason for hiding this comment

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

发现一个替代品: https://titlecaseconverter.com/

Copy link
Contributor Author

@AlexShan2008 AlexShan2008 Jan 15, 2025

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.

考虑到这是一个线上的问题,建议先修复。后续进一步讨论具体的例子,示例不仅要求正确并且最好具有代表性。

@rainx rainx merged commit 29100a0 into main Jan 16, 2025
2 checks passed
@github-actions github-actions bot locked and limited conversation to collaborators Jan 17, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants