mozilla-japan/modest_themes — pull request #5 by ethertank (2013-03-08 10:25:27+00:00)
Title: カラム落ちの修正など

カラム落ちなどを修正しました。
カラム落ち修正については実質 3 行、コメント欄前で左カラムを閉じている余計な〆div の削除と、php 内の endif の位置の修正です。

完全に修正出来たと思いますが、記事データ等はいただいておらず、完全なプレビューは行っておりません。各種ページの確認をお願いしたいと思います。おそらくコメントの有無によってカラム落ちの発生の有無も変わっていたのではないかと思います。その点も含め、ご確認いただければ幸いです。

その他、色々変更してしまいました。
問題があればカラム落ちの修正のみで再度投稿致します。
以下に変更点等をリストアップします。

---

#PHP
- 問題点等を探す為のみに詰め / 展開
- 一部 HTML 階層に誤解を生じ得るインデントを修正
- 出力に影響を与える改行を一部 PHP タグ内に移動（h3 > a 間等）。
- 修正: パーマネントエリアで特定条件時にのみ発生するカラム落ちの修正。endif の位置によるものか。修正を試みた際に追加されたと思しき（勘）不要な /div を一つ削除。

#CSS
- 確認: 問題点等を探す為のみに一旦インデント、改行 / カンマ後にスペースを追加
- 補完: :hover にスタイルが当てられており :focus に無いものを補完
- 後方互換性: gradient を含む background ショートハンドを分解し背景色がフォールバックとして機能するようにした
- 後方互換性: 一箇所のみ rgba から rgb カラーに変更、レンダリング負荷軽減も兼ねる
- 前方互換性: gradient(top... -> gradient(to bottom...
- 簡略化: \#FFFFFF -> \#FFF
- 簡略化: 10px 10px -> 10px
- 簡略化: 0px, 0em, 0deg -> 0 /* 0 値の単位を削除 */
- 簡略化: -webkit-gradient color-stop(xxx 0%/100%) -> from/to(xxx)
- 簡略化: 0.5px と .5px の混在を後者に統一
- 修正: 重複スタイルを削除。変更後、問題は発生していないように思われる
- 修正: transition プロパティに於いて、float 及び display プロパティに対する指定は無効 -> 削除
(※opacity に対しては可能。よって、display で意図した（と思われる）動作は opacity で実現可能。)

#HTML
- コピーライト部文の年を 2013 に変更
- 一部見出しの不正なジャンプを修正し、セクショニングを改良（フッター見出しがやや大きくなるが、そのままとした）
- ログイン時にのみ表示されるボタン(a 要素)をラップしている p 要素を div 要素に変更（上下マージンが変わるが、そのままとした）
- アバター部分の address 要素を div 要素に変更（スタイル変わらず）
- コピーライト部分をセマンティクス付与の目的で、small でラップ（文字が小さくなったが、調整は行っていない。調整すべき）

#他
- gitignore: Thumbs.db を除外リストに追加
- 画像: 無劣化圧縮を適用
- html5.js(html5 shiv) の更新

#改変しなかった部分 / 今後改良すべき部分
- CSS プリフィックス -o- の削除
- トップページへの導線がロゴ部分のリンクのみ。「このサイトについて」がサイドバーに常時表示されているので、グローバルナビ部分は「ホーム」に変更しても良いかもしれない。ウィンドウサイズ変更によりナビの一部が隠れるスタイルが適用されているので、変更するならばここも同時に行う必要がある（読者層的にこのままの方が良い気もする）
- 表示速度向上やコード量削減の余地もあるが、行っていない。
- font-weight: bold / normal の 700 / 400 への変更は行っていない。
- input[type="text"]、サイドバーのフォントが他部位と異なるので統一してみたが、あまり良くなかった。
- 数カ所のセマンティクスに違和感