Autolink of timeline message

kannokanno commented 11 years ago

メッセージにhttp(s)が含まれていればリンク化するようにしました

haushinka2dx commented 11 years ago

他は特に指摘ありません。

kannokanno commented 11 years ago

theme.cssに移しました。とくにrebase -iしていません(別々のコミットとなっている)

コミットをまとめた方がよければ言ってください。

haushinka2dx commented 11 years ago

コミットまとめなくて良いと思います。後ほど動作確認して取り込みます。

kannokanno commented 11 years ago

はいー

haushinka2dx commented 11 years ago

動作はバッチリですが、デザインに一言だけ。

アンダーラインで青文字になってますが、水色背景に青文字だと見難い感じがします。文字列見ればリンクだって分かるしカーソル当てればカーソル変化するので、

.timeline-item-message a {
// text-decoration: underline;
    color: #ffffff;
}

.timeline-item-message a:link {
//  color: #0000C4;
}

って感じで

でもUXは下がらないような気がしますがどうでしょうか。

kannokanno commented 11 years ago

やっぱ見づらいですか。そこ悩みました。

という判断で今のデザインにしました。(青/紫が見にくいというのはありますが) が、以下の点で微妙というかデメリットもあるなと感じています。

ユーザーに与える情報は最小限の方がいい(たくさん情報を与えても混乱させるだけ)
- -> リンクに関心のあるユーザーだけが、それがクリック可能であることを知れればいい
- = カーソル変化だけでいいのでは
「過去に〜」の場合のことはそれが必要になってからデザイン再検討すればいいのではないか

ということで、上記案でよさそうと思いますので変えまっす

kannokanno commented 11 years ago

変えましたん

haushinka2dx commented 11 years ago

どもです！後でマージしてリリースします。

haushinka2dx / atmosphere-web