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

3-4レビュー修正案 #127

Merged
merged 90 commits into from
May 11, 2021
Merged

3-4レビュー修正案 #127

merged 90 commits into from
May 11, 2021

Conversation

bakera
Copy link
Owner

@bakera bakera commented Mar 8, 2021

全体的に内容モデルの話、a11yの話、その他いろいろ追加しています。

Copy link
Collaborator

@momdo momdo left a comment

Choose a reason for hiding this comment

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

暫定的にレビュー内容を反映させる方向で

md_text/3-4.md Outdated Show resolved Hide resolved
md_text/3-4.md Outdated

`download`属性が存在する場合、`href`属性で示されたリソースをダウンロードします。`download`属性の値には、ダウンロード時のファイル名を指定します。属性の値に制限はありませんが、この属性の値が必ず使われるとは限りません。OSの制約でファイル名に使用できない文字は、削除されたり置換されたりすることがあります。また、実際にダウンロードする際のHTTP応答ヘッダでファイル名が指定されていた場合は、属性の指定よりもそちらが優先されます。
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
`download`属性が存在する場合、`href`属性で示されたリソースをダウンロードします。`download`属性の値には、ダウンロード時のファイル名を指定します。属性の値に制限はありませんが、この属性の値が必ず使われるとは限りません。OSの制約でファイル名に使用できない文字は、削除されたり置換されたりすることがあります。また、実際にダウンロードする際のHTTP応答ヘッダでファイル名が指定されていた場合は、属性の指定よりもそちらが優先されます。
`download`属性が存在する場合、`href`属性で示されたリソースをダウンロードします。`download`属性の値には、ダウンロード時のファイル名を指定します。属性の値に制限はありませんが、この属性の値が必ず使われるとは限りません。OSの制約でファイル名に使用できない文字は、削除されたり置換されたりすることがあります。また、実際にダウンロードする際のHTTP応答ヘッダーでファイル名が指定されていた場合は、属性の指定よりもそちらが優先されます。

応答をレスポンスにしたほうがよいか?

md_text/3-4.md Outdated Show resolved Hide resolved
md_text/3-4.md Outdated Show resolved Hide resolved
md_text/3-4.md Outdated Show resolved Hide resolved
md_text/3-4.md Outdated Show resolved Hide resolved
md_text/3-4.md Outdated Show resolved Hide resolved
md_text/3-4.md Outdated Show resolved Hide resolved
md_text/3-4.md Outdated Show resolved Hide resolved
md_text/3-4.md Outdated Show resolved Hide resolved
@momdo momdo merged commit 4cf0f99 into master May 11, 2021
@momdo momdo deleted the review3-4 branch May 11, 2021 02:33
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