search

osyo-manga / vital-over

18 stars 3 forks source link

Module.KeyMapping: fix <SID> handling #113

Closed haya14busa closed 9 years ago

haya14busa commented 9 years ago

maparg() の rhs は <SID>のまま入っていたのでそこをちゃんと変換するようにしました.

palette でやるべきな気もする(けど, paletteは maparg()の返り値を返して欲しいという思いもある)のでそのへんはおしょーさんの意見を伺いたいです

osyo-manga commented 9 years ago

ありがとうございます。 個人的には s:_convert_sid() 相当の機能を vital-palette に追加したほうがよいかなーという感じですかね。 （既存の機能はそのままでコンバータだけ用意する感じ。

haya14busa commented 9 years ago

Palette にそのまま移植して, それを使うようにしました

haya14busa commented 9 years ago

これもどちらかというとバグの部類なのでなんらかの手段で対応したさありますが, これって何がダメなんでしたっけ...(忘れてしまった)

haya14busa commented 9 years ago

こっちの実装場所の問題か https://github.com/osyo-manga/vital-palette/pull/2

osyo-manga commented 9 years ago

すみません、完全に忘れてました。 いまだに vital-palette の方の仕様どうするか決めかねてるのでとりあえずこっちに取り込んじゃいたいと思います。

osyo-manga commented 9 years ago

ありがとうございました。

haya14busa commented 9 years ago

すみません、完全に忘れてました。 いまだに vital-palette の方の仕様どうするか決めかねてるのでとりあえずこっちに取り込んじゃいたいと思います。

ｱｱｱｰ... このブランチpalette対応のｺﾐｯﾄたしてるのでこのままﾏｯｼﾞは動かないフラグです

osyo-manga commented 9 years ago

途中でコードが変わっていた力…

osyo-manga commented 9 years ago

手動であれします。

haya14busa commented 9 years ago

手動であれします。

すいませんよろしくお願いしますっ