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

Change required description on Safari #512

Merged
merged 1 commit into from
May 29, 2018

Conversation

wudifeixue
Copy link
Contributor

In this PR:

Fixes #447

Copy link
Contributor

@S1ngS1ng S1ngS1ng left a comment

Choose a reason for hiding this comment

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

Typo :P

@@ -1337,7 +1337,7 @@
"例如,如果你想把一个文本输入字段设置为必填项,在你的<code>input</code>元素中加上<code>required</code>属性就可以了,你可以使用:<br><code>&#60;input type=\"text\" required&#62;</code>",
"任务:给你的输入框添加<code>required</code>属性,这样用户不填写输入框就无法提交表单。",
"试试输入框不填写任何文本,然后点击Submit提交表单,看看浏览器如何提示你该字段为必填项。",
"注意:<code>required</code>属性在Safari浏览器中不起作用,请用其他浏览器来学习,推荐使用Chrome。"
"注意:<code>required</code>属性在Safari浏览器10.1以下不起作用(2017年3月27以前),请更更新Safari或者使用其他浏览器来学习,推荐使用Chrome。"
Copy link
Contributor

Choose a reason for hiding this comment

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

"请更更新" => "请更新"

Copy link
Contributor

@S1ngS1ng S1ngS1ng May 18, 2018

Choose a reason for hiding this comment

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

Also, please unify the symbols (parenthesis here) to EITHER full-width OR half-width. Mix & match won't be a good idea 😂

Copy link
Contributor Author

Choose a reason for hiding this comment

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

哈哈好尴尬呀居然打错了字,那我括号也改一改好了

@wudifeixue wudifeixue force-pushed the fix/safari-compatibility-description branch from b91d045 to d14c6be Compare May 28, 2018 18:23
@wudifeixue
Copy link
Contributor Author

Updated now

@S1ngS1ng S1ngS1ng merged commit ebccaa2 into dev May 29, 2018
@wudifeixue wudifeixue deleted the fix/safari-compatibility-description branch June 25, 2018 02:23
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.

2 participants