sakura
sakura copied to clipboard
スクロールの行数の設定インターフェースについて考える
スクロールの行数の設定インターフェースについて考える
#259 の件で スクロールの行数を試しに 20 に設定して見たのだが設定が保存されなかった。
手順
-
設定
を選ぶ -
共通設定
を選ぶ -
スクロール
の行数
を20
と入力する -
OK
を押してダイアログをもう一度開く -
行数
の設定が 10 に戻っている。
以下のコードで切り詰められている模様
https://github.com/sakura-editor/sakura/blob/757d43bbd1dec174de540aabb25c287db874ad81/sakura_core/prop/CPropComGeneral.cpp#L203-L209
20 に設定したつもりでも、実際には反映されていないのでバグっぽく見える。
スピンコントロールというかアップダウンコントロールですね。
上限下限を指定してあげれば x < 1 or 10 < x の値を設定できないようなります。 テキストボックスの入力文字数制限は結構やってますので追加しても害はないと思います。 独自関数UpDown_SetRangeが定義されているのでこれを使って範囲指定すればよいはず。
実装例↓ https://github.com/sakura-editor/sakura/blob/757d43bbd1dec174de540aabb25c287db874ad81/sakura_core/dlg/CDlgWinSize.cpp#L99
ちなみにこの設定、windowsのコントロールパネル -> マウス -> ホイールにある縦スクロールの行数と内容が被るんですよね。元の話がどんなときのことを言ってるか微妙だったんであれですが、マウススクロールに関してはwindowsの設定とここの設定の「両方を見る」という微妙な仕様です。
ユーザーエクスペリエンス的なことを考えると、設定項目自体を廃止する選択もあると思っています。
ユーザーエクスペリエンス的なことを考えると、設定項目自体を廃止する選択もあると思っています。
これがいいのかも。
#259 上下カーソルキーの移動間隔がスクロールの行数になっているのを変えませんか?
にも色々書きました。
ユーザーエクスペリエンス的なことを考えると、設定項目自体を廃止する選択もあると思っています。
これがいいのかも。
これで行くとして、誰が実装して、誰がレビューするんや?という課題があります。