色キーワードに関するコードの整理

• Reporter: revulo
• Owner: dhrname
• Type: パッチ報告
• Status: Closed
• Component: (None)
• MileStone: (None)
• Priority: 5 - Medium
• Severity: 5 - Medium
• Resolution: Fixed
• File: 1

Details Reply

var colorsname = {...} という部分のコードを見直してみまし
た。書き換えた分のコードを添付しておきます。

・改行と空白を入れて分かりやすく

・不要な "" を削除

・darkgoldenrod の値が "#B886B" になっていたの
を "#B8860B" に修正

・VML がネイティブにサポートしている 16 色分の定義をコメント
アウト

・グローバル変数だったのを、NAIBU.PaintColor の名前空間の中
に移動

・SVG の仕様書には color keywords と書かれているので、それに
合わせて連想配列の名前を colorsname から keywords に変更

VML がサポートしている16色については定義しなくても動作するの
で、軽量化を目的に削ってみました。コメントアウトで残してあるの
は、定義し忘れではないというメモのつもりですが、邪魔でしたら、
コメントを外すか、コメントの行ごと削ってしまっても構いません。

それと、このコードに置き換える場合は、NAIBU.PaintColor
= ... の定義よりも後ろに書いて下さい。そうしないとエラーになっ
てしまうようです。

Ticket History (3/6 Histories)

2008-09-19 01:56 Updated by: revulo

• File 3444: sie_colorsname.js is attached

2008-09-19 19:26 Updated by: dhrname

• Resolution Update from None to Accepted
• Owner Update from (None) to dhrname
• Priority Update from 5 - Medium to 8

Comment Reply

Logged In: YES
user_id=32883

パッチを送信していただきありがとうございました。送られたパッチはSIEと同じライセンス（すなわち、MPL1.1やGPL、LGPL)で提供されたものとみなしますので、ご注意ください。

質問、意見等があれば、下記のメーリングリストまで。
http://lists.sourceforge.jp/mailman/listinfo/sie-developers

2008-09-19 23:19 Updated by: dhrname

• Priority Update from 8 to 5 - Medium

Comment Reply

Logged In: YES
user_id=32883

ありがとうございました。非常に助かります。
コメントの行ごと削って、リビジョン497で当てまし
た。

2008-09-27 20:24 Updated by: dhrname

• Priority Update from 5 - Medium to 4
• Resolution Update from Accepted to Fixed

2008-10-14 19:57 Updated by: dhrname

• Priority Update from 4 to 5 - Medium

2008-10-14 19:57 Updated by: dhrname

• Ticket Close date is changed to 2008-10-14 19:57
• Status Update from Open to Closed