search

sakura-editor / sakura

SAKURA Editor (Japanese text editor for MS Windows)
https://sakura-editor.github.io/
Other
1.23k stars 162 forks source link

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

Open m-tmatma opened 6 years ago

m-tmatma commented 6 years ago

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

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

手順

scroll

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

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

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

berryzplus commented 6 years ago

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

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

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

berryzplus commented 6 years ago

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

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

KENCHjp commented 6 years ago

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

これがいいのかも。

berryzplus commented 3 years ago

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

にも色々書きました。

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

これがいいのかも。

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