na2hiro / Kifu-for-JS

JavaScriptで動く将棋の棋譜再生盤とそのブックマークレット `kifu-for-js`、JSON棋譜フォーマット(JKF)の定義とKIF, KI2, CSAからの変換ライブラリ `json-kifu-format`、将棋のモデル`shogi.js`を含むmonorepo
MIT License
112 stars 16 forks source link

持駒の歩が2-3枚ある場合の表示不具合 #34

Closed mizar closed 7 years ago

mizar commented 7 years ago

http://wdoor.c.u-tokyo.ac.jp/shogi/LATEST/2017/06/21/wdoor+floodgate-300-10F+gpsfish_normal_1c+yrz7_Cortex-A53_1c+20170621213011.csa

上記棋譜において、 20手目:後手の持駒が「歩2」→「角歩2」なのに表示が「角歩」 29手目:先手の持駒が「飛金桂歩3」、歩の表示が縦にはみ出す などの不具合があるように見受けられます。

持駒の歩の枚数が2-3枚の場合は縦に並べる、という部分が混乱の元になっているように見受けられるので、このような場合にも歩は横に並べるようにしては如何でしょうか。

mizar commented 7 years ago

https://github.com/orangain/Kifu-for-JS/commit/9c32d07a66e800be50736960a5d419899c77d9dd にて、この問題の回避は可能であるようです。

orangain commented 7 years ago

33 をマージいただいたので、この問題のPR出しておきます。

na2hiro commented 7 years ago

@mizar ご報告ありがとうございました.どうも最近の変更で入ったバグのようですね. @orangain 迅速にPRを作っていただきありがとうございました.