Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

キーボードフォーカスが当たっていないテキストボックスの文字列が選択状態になっていることがある #1717

Closed
yoshinrt opened this issue Aug 27, 2021 · 6 comments

Comments

@yoshinrt
Copy link
Contributor

問題内容

どこかの段階で仕様変更があったのでしたらご容赦ください.

Grep 置換ダイアログや,ツールバーの検索ボックス (設定で追加) に過去履歴文字列が現れるとき,キーボードフォーカスが当たっていないにもかかわらず,文字列が選択状態になっています.
他にあるかは未確認です.

スクリーンショット 2021-08-27 202647

期待する動作

  • 過去リビジョンに合わせるなら,未選択
  • 新しく選択状態の仕様になったのでしたら,灰色選択
    • ツールバーの選択状態は目立つので,できれば未選択が希望です

再現手順

  1. Grep 置換で何か置換を実行する
  2. 再び Grep 置換ダイアログを表示すると,置換前 と 置換後の両方が選択状態

または

sakura 起動直後の状態で,ツールバーに足した検索ボックスの文字列が選択状態

再現頻度

100%

問題のカテゴリ

  • プログラムの動作上の問題

環境情報

  • OS バージョン
    Win10 Pro 64bit

  • サクラエディタバージョン

サクラエディタ   v2.4.2.1160 32bit dev
(GitHash 2deb79197c608f00897453a762d9178d07fc59e5)
(GitURL https://github.com/sakura-editor/sakura)

      Compile Info: V1929 WPR WIN601/I800/C000/N601
      Last Modified: 2021/7/31 21:13:37
@berryzplus
Copy link
Contributor

趣旨は了解です。

キャプチャの「Grep置換」ダイアログが標準と異なりますね・・・。

Windows側で変則DarkModeにしているんだと思いますが、
ここのプロジェクトはまだその域に達しとらんので「へ?」と思いました。

@yoshinrt
Copy link
Contributor Author

git bisect してみました.直接の原因ではないかもしれませんが.
Grep ダイアログとツールバーは別のところで挙動の差分が出ました.

Windows側で変則DarkModeにしているんだと思いますが、

タイトルバーの色のことでしたら,Win10 標準の色を変える機能です.(個人用設定→色)
sakura の挙動に関わる変な物は使ってない... はず...

@usagisita
Copy link
Contributor

「外部コマンド実行(ファイル名を指定して実行)」の「カレントディレクトリ」でも同様の現象が見られます。
カレントディレクトリのチェックが外れていてグレーダウンしているときも背景の青い表示になります。
なお当方の環境はWin8.1です。
2.3.0.1ぐらいのときは選択状態ではなかったと思います。

@beru
Copy link
Contributor

beru commented Aug 29, 2021

「外部コマンド実行(ファイル名を指定して実行)」の「カレントディレクトリ」でも同様の現象が見られます。
カレントディレクトリのチェックが外れていてグレーダウンしているときも背景の青い表示になります。
なお当方の環境はWin8.1です。
2.3.0.1ぐらいのときは選択状態ではなかったと思います。

こちらの環境 (Windows 10 バージョン 20H2) ではこの現象は見られませんでした。

実装内容を見てみたところ、CDlgExec::SetData でコンボボックスのアイテムを追加しているので、OnCbnDropDown メソッドを追加して、まだコンボボックスに要素が追加されていない場合(初回)にアイテムを追加するような対策で解消するかもしれないですね。

@usagisita
Copy link
Contributor

追加で調査したところ、普段使いの少し古いバージョンでは発生し、master/HEAD最新版では発生しないらしいことが分かりました。
手元のバイナリで残ってるものとbisectで探した感じでは、
aa615db

  • 印刷ページ設定ダイアログのフォントが巨大化する問題を修正
    このコミットで修正されたようです。
    お手数おかけしました。

@beru
Copy link
Contributor

beru commented Sep 8, 2021

#1719#1724 のMergeにより報告された問題は解消したと思うので Close します。

@beru beru closed this as completed Sep 8, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants