sakura icon indicating copy to clipboard operation
sakura copied to clipboard

スクロールの行数の設定インターフェースについて考える

Open m-tmatma opened this issue 6 years ago • 4 comments

スクロールの行数の設定インターフェースについて考える

#259 の件で スクロールの行数を試しに 20 に設定して見たのだが設定が保存されなかった。

手順

  • 設定 を選ぶ
  • 共通設定 を選ぶ
  • スクロール行数20 と入力する
  • OK を押してダイアログをもう一度開く
  • 行数 の設定が 10 に戻っている。

scroll

以下のコードで切り詰められている模様

https://github.com/sakura-editor/sakura/blob/757d43bbd1dec174de540aabb25c287db874ad81/sakura_core/prop/CPropComGeneral.cpp#L203-L209

20 に設定したつもりでも、実際には反映されていないのでバグっぽく見える。

m-tmatma avatar Jul 16 '18 12:07 m-tmatma

スピンコントロールというかアップダウンコントロールですね。

上限下限を指定してあげれば x < 1 or 10 < x の値を設定できないようなります。 テキストボックスの入力文字数制限は結構やってますので追加しても害はないと思います。 独自関数UpDown_SetRangeが定義されているのでこれを使って範囲指定すればよいはず。

実装例↓ https://github.com/sakura-editor/sakura/blob/757d43bbd1dec174de540aabb25c287db874ad81/sakura_core/dlg/CDlgWinSize.cpp#L99

berryzplus avatar Jul 16 '18 13:07 berryzplus

ちなみにこの設定、windowsのコントロールパネル -> マウス -> ホイールにある縦スクロールの行数と内容が被るんですよね。元の話がどんなときのことを言ってるか微妙だったんであれですが、マウススクロールに関してはwindowsの設定とここの設定の「両方を見る」という微妙な仕様です。

ユーザーエクスペリエンス的なことを考えると、設定項目自体を廃止する選択もあると思っています。

berryzplus avatar Jul 16 '18 14:07 berryzplus

ユーザーエクスペリエンス的なことを考えると、設定項目自体を廃止する選択もあると思っています。

これがいいのかも。

KENCHjp avatar Jul 16 '18 20:07 KENCHjp

#259 上下カーソルキーの移動間隔がスクロールの行数になっているのを変えませんか?

にも色々書きました。

ユーザーエクスペリエンス的なことを考えると、設定項目自体を廃止する選択もあると思っています。

これがいいのかも。

これで行くとして、誰が実装して、誰がレビューするんや?という課題があります。

berryzplus avatar Sep 22 '20 15:09 berryzplus