r-labs » Work Time

良い機能だと思います。

ただ、画面変更のリンクの隣にあると、画面遷移の一つだと思われそうなので、
別の場所にポタンで作る(「何か実行しちゃうよ」と感じる)のが良いと思うのですが、どうでしょうか。

そういう意味では、既存の機能もリンク表示のものがあるので、統一した方が良いと思うのですが、
別チケットでの対応で良いと思います。

ただ、画面変更のリンクの隣にあると、画面遷移の一つだと思われそうなので、
別の場所にポタンで作る(「何か実行しちゃうよ」と感じる)のが良いと思うのですが、どうでしょうか。

なるほどなるほど。
ちょっとアップデートしてみました。

https://bitbucket.org/suvene/redmine_work_time/commits/28f05e3cd09ff59c413bd395d7abae0c1fbd0462

派手すぎるかな？
twitter bootstrap の css 使ってます。

そういう意味では、既存の機能もリンク表示のものがあるので、統一した方が良いと思うのですが、
別チケットでの対応で良いと思います。

refs #1195

はや！
すごすぎです。ww

やっぱりちょっとハデかもしれないですね。

ボタンであれば「実行」をイメージできると思うので、redmine本体のスタイルと同じでいいかも。
私の表現(実行を感じる)が良くなかったですね。すみません。

ボタンの位置ですが、リストの右上and/or右下、または左下が良い気がするのですが、どうでしょうか？

Tomohisa Kusukawa は書きました:

やっぱりちょっとハデかもしれないですね。
ボタンであれば「実行」をイメージできると思うので、redmine本体のスタイルと同じでいいかも。

了解です。

ボタンの位置ですが、リストの右上and/or右下、または左下が良い気がするのですが、どうでしょうか？

このイメージの、1 and/or 2, or 3 みたいな感じですか？

画面の右側に配置しなかったのはわざとです。しょっちゅう押す機能ならまだしも、「月に一度押すか押さないかのボタンなのに、しょっちゅう操作する近くに配置するのはいかがなものか？」みたいな根拠で。なので、3ですかね?

このイメージの、1 and/or 2, or 3 みたいな感じですか？

そのとりです。
3に賛成です。

たぶん、ボタン位置などにブレがないと思ったので、fix しました。

ホントは同じブランチで対応するのはよくないんですが、ajax を通すと、<td> タグが二重になってしまう不具合に対応してしまいました。

おっと。そんな不具合があったとは。
修正、ありがとうございます。