「指定行へジャンプ」ダイアログで入力できる桁数を増やす

[beru]

PR の目的

「指定行へジャンプ」ダイアログで行番号に入力できる桁数を増やすのが目的です。

カテゴリ

• 仕様変更

PR の背景

#1827 で報告された問題に対処をしています。

PR のメリット

ファイルの行数が6桁より大きいファイルで指定行へジャンプ出来るようになる。

PR のデメリット (トレードオフとかあれば)

「指定行へジャンプ」ダイアログの行番号入力箇所の横幅が広くなるので、桁数が小さい場合に広すぎるように感じる。

仕様・動作説明

仕様変更前は行番号入力箇所に6文字までしか入力できませんでした。

仕様変更後は行番号入力箇所に9桁までの数値を入力する事が出来ます。

PR の影響範囲

「指定行へジャンプ」ダイアログのみ

テスト内容

テスト1

手順

• 行数が多いファイルを開く
  • very_long_file.7z.zip
    • 拡張子をzipに偽装した7zファイルです。
• 「指定行へジャンプ」ダイアログを開く
• 行番号入力箇所に 12345678 と入力してジャンプボタンを押す。
• 12345678行目に移動する。

関連 issue, PR

#1827

[sonarcloud]

Kudos, SonarCloud Quality Gate passed!   

0 Bugs
0 Vulnerabilities
0 Security Hotspots
0 Code Smells

No Coverage information
No Duplication information

[AppVeyorBot]

✅ Build sakura 1.0.4102 completed (commit 638a6adbb8 by @beru)

[usagisita]

AppVeyorのexeをダウンロードしてきて実行してみて、9桁ではなく8桁までは入力できることを確認しました。
もしかしたら9桁以上入力したい人もいるかもしれませんが……
LGTMだと思います。

[berryzplus]

6桁しか入力できない問題に対して、入力できる桁数を増やして対応することには自分も賛成です。
どこまで増やすか？＝新しい上限をどうするか？についても8桁あれば困らないような気がします。

intの最大値は 2,147,483,647 で 10桁 なので10桁にしたい人が出てくるかも知れませんが、
その時は「8桁までしか入力できない」のissueをあげてもらえれば良いと思います。

[beru]

レビューありがとうございました。Mergeします。
もし問題が見つかったらまた別のPRで修正します。