mozilla-japan / dev.mozilla.jp

dev.mozilla.jp のテーマ
1 stars 1 forks source link

ボタン部分を見出し要素からリストアイテムに変更 #18

Closed ethertank closed 11 years ago

ethertank commented 11 years ago

文書構造に関わる軽微な修正です。見出し要素からリストアイテムへ変更し、スタイルを追加しています。

ボタン間の垂直マージンは 1em とし、従来よりややコンパクトに。全レイアウトで目視確認済みですが、問題は無いと思います。

tetsuharuohzeki commented 11 years ago

本テーマの開発時に「ユーザーの参加を促したい」という狙いがあって、それを少なからず反映させたものとして見出しにしていました。

ですが、nav 要素の子孫に含まれている時点でその目的は果たされたものとも解釈できます。必要とあらば、今後、section などでくるむなどの措置の導入を考えてもいいかもしれません。

@dynamis アウトラインを変更して大丈夫ですか?

ethertank commented 11 years ago

見出し要素はあくまで見出し要素であり、強調の目的には適しません。もしテキストの重要性を示す意図があるのなら strong が適切ですが、ここではそれも必要ないと考えます。美しいボタンのスタイルで視覚上の訴求力もあるので、自然なマークアップで良いと思います。

nav 要素が既にセクションを切っており、直後に匿名セクションがあるという状態です。section でのラップも必要無いでしょう。「このサイトについて」の部分も対応する内容を持たず、見出し要素でのマークアップは不適切だと考えます。この部分や他部位のセマンティクス、アウトラインに関しても改善を予定しておりますので、お任せいただければ幸いです。

tetsuharuohzeki commented 11 years ago

了解しました。

この部分や他部位のセマンティクス、アウトラインに関しても改善を予定しておりますので

その件については、該当のコミット/プルリクエストを記録していくissueを立ててもらえますか?

tetsuharuohzeki commented 11 years ago

https://github.com/mozilla-japan/modest_themes/issues/20 メタバグの参照

ethertank commented 11 years ago

2 件に分けて issue を立てました。こんな感じで宜しいでしょうか?

前者については現在少々問題があり、特に重視しています。物理構造がしっかりしているのでそれほど手はかからないと思いますが、サイドバー部分は将来どういった変更の可能性があるのかこちらが把握できていないので、あまり大胆な事は出来ません。今の所、ボタン部分と他部位のセクションを切り離し、柔軟性を他部位に委ねるのが良いかなと考えています。

後者については、意味的なタグの追加や変更がメイン。イベントページのリッチスニペットなどにも手を付けられれば尚良しと考えています。 http://support.google.com/webmasters/bin/answer.py?hl=ja&answer=164506

tetsuharuohzeki commented 11 years ago

2 件に分けて issue を立てました。こんな感じで宜しいでしょうか?

  • 20 アウトライン

  • 21 その他のセマンティクス

はい。それで問題ないです。 新規にissueまたはpull requestを行う時に、上記2つのissueのどちらかのURLをコメント欄に参考として貼っておく事で、自動的に上記のissueに参照された旨が表示されます。

サイドバー部分については、当面は現在のままです。将来的にプロジェクトが増える事があれば、アコーディオン化などが必要になりますが、、、かなり遠い未来だと思います。

scheme.org etcへの対応は開発時より課題として上がっていましたが、人手の面で断念していました。

色々とお手数をおかけします<( )>

ethertank commented 11 years ago

自動的に上記のissueに参照

なるほど。了解しました。

当面今の構造ということであれば、サイドバーに関しては時間はかからないと思います。

カラム落ちだけ気になって手を出したら楽しくなってきたという感じなので、お手数なんてとんでもないです。 コミットの練習みたいになってしまって申し訳ありませんが、もう少々お付き合いいただければ幸いです。