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

代替テキストの決定ツリーを追加 #141

Closed
wants to merge 2 commits into from

Conversation

syasuda90
Copy link
Contributor

@syasuda90 syasuda90 commented Nov 26, 2020

チェック項目

Pull Request を出す前に確認しましょう

  • Pull Request の概要を適切に書いた
  • レビュアの指定をしている
  • textlintを実行しエラーが出ていない

概要

こちらのissue にある代替テキストの付け方の説明を追加するの対応を行いました。
エイヤという感じでマークアップしたのでまだ草案 or モックレベルという温度感ではあります。
ですのでいくつか相談しつつ作り上げていきたい所存です。

決定木の項目等に関してはこちら を参考(許可を得て)にしつつAmebaライクなものに仕上げていこうと思っています。

スクリーンショット

スクリーンショット 2020-11-26 16 00 38

このような形で現在は 1.1.1 画像に代替テキストを提供する の最下部に配置しています。

マージするまでに達成すること

  • 決定ツリーの見た目Fix
  • 決定ツリーをどこに配置するか
  • css(class名など)やhtmlの配置に関するルールが適切に対応できているか

@syasuda90 syasuda90 self-assigned this Nov 26, 2020
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.

1 participant