Feature #188: メソッド名のリンクをもっと目立たせる。

こんばんは sheepman です。

シグネチャに下線を引いたりすると見にくくなると思うので、
マウスオーバーで td 全体の色が変わるか、メソッドの説明に「続きを読む」を
表示するのが良いと思います。

On Wed, 18 Jun 2008 19:19:33 +0900
Yutaka HARA <[e-mail]> wrote:

> チケット #188 が報告されました。 (by Yutaka HARA)
> 
> ----------------------------------------
> Feature #188: メソッド名のリンクをもっと目立たせる。
> http://redmine.ruby-lang.org/issues/show/188
> 
> 起票者: Yutaka HARA
> ステータス: Open
> 優先度: Normal
> 担当者: 
> カテゴリ: bitclust
> Target version: 
> reporter: 
> ruby_version: 
> 
> 
> 各クラスのメソッド一覧で、現状だとメソッド名にしかリンクが張られていないので、
> 初めて見た人はそこがクリックできることに気がつかないらしい。
> (複数人から報告を受けました^^;)
> 
> リンクのCSSを変えるか、メソッドのシグネチャもリンクに含めるか、<td>全体をクリックできるようにするなどしてリンクを目立たせるといい。
> 
> 

-- 
sheepman / TAMURA Takashi

--
ML: [e-mail]
使い方: http://QuickML.com/

okkez です。

あんまり HTML とか CSS には詳しくないのですが、ちょっと試してみました。

最後に付けたパッチでマウスオーバーの際に td の背景色は変わりますが、リンクは以前のままです。

a { display: block; }

とかやれば td の内部が a タグのみの場合は td 全体にリンクが付くのですが、今回の場合はちょっと
厳しそうです。
bitclust のソースもちょっとだけ見てみましたが、リンクがメソッド名にしか付かないので
上記の方法を実現しようと思うと、ちょっと本体に手を入れる必要がありそうです。

yhara さん案でも sheepman さん案でも、bitclust に手を入れる必要があるのかなと思いました。

# td の onClick を使って javascript でリンクを実現する手もあるかな。

Index: theme/default/style.css
===================================================================
--- theme/default/style.css     (revision 2793)
+++ theme/default/style.css     (working copy)
@@ -186,6 +186,11 @@
     width: 20em;
 }

+td.signature:hover {
+    color: #fff;
+    background-color: #4aa;
+}
+
 td.description {
     padding: 0.5em;
     background-color: #EEE;



-- 
okkez
[e-mail]

--
ML: [e-mail]
使い方: http://QuickML.com/

• Due date set to 08/13/2008
• Assignee set to aamine (Minero Aoki)

幅は広げてみたが、高さが<td>の100%に広がらない。

• File css.patch added

メソッド一覧，クラス一覧などのリンクの表示を改善するパッチを添付します。
これで，a 要素が td の天地左右いっぱいまで占めるようになります。
ようはブロックレベルにした a 要素を height: 100% にすればいいわけですが，padding が 0 でないと，そのぶん td をはみ出してしまいます。
そこで，a は padding: 0 にしておいて，その中に span を入れて，その span を display: block; padding: 0.3em にしました。
また，a の中に code 要素が入っている場合はその code を display: block; padding: 0.3em にしました。
a の中に span を入れるため，style.css だけでなく a_href メソッドも書き換えました。

なお，このパッチは Bug #55 のパッチを含んでいます。
また，横の説明文のところで，td の内余白の天地が無駄に空いているように思えたので，ちょっと狭くしてみました。

すみません，この css.patch は取り下げます。
よく確認したつもりでしたが，このやり方ではうまくいっていませんでした。
CSS で a を td の天地いっぱいにするのは無理なような気がします。
JavaScript を使えば簡単ですが…。
ゴミを投稿してしまって申し訳ありませんでした。

• File deleted (css.patch)