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

ウィンドウ一覧画面を開いた後にウィンドウサイズを大きくしてから閉じて開き直すとウィンドウサイズにコントロールが追従していない問題を修正 #726

Merged
merged 1 commit into from
Jan 2, 2019

Conversation

beru
Copy link
Contributor

@beru beru commented Dec 26, 2018

問題の再現手順はタイトルに書いた通りです。

なお、ウィンドウ一覧画面は #687 にも関係しています。そちらの確認をしていてこの問題に気付きました。

@beru beru added the 🐛bug🦋 ■バグ修正(Something isn't working) label Dec 28, 2018
Copy link
Contributor

@berryzplus berryzplus left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

コード的に問題なさげです。
実際どう動くかの確認して大丈夫そうなら入れてよいと思います。

@beru
Copy link
Contributor Author

beru commented Dec 30, 2018

@berryzplus さん

コード的に問題なさげです。
実際どう動くかの確認して大丈夫そうなら入れてよいと思います。

お手数ですが動作確認お願いします。

Copy link
Contributor

@berryzplus berryzplus left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

動作確認したところ、問題なさそうでした。

再現手順

  1. メニュー [ウインドウ]⇒[ウインドウ一覧表示]
    ※このPR適用前、画面解像度(≒表示倍率)によってはこの時点で既におかしい
  2. ウインドウ一覧画面を縦方向に大きくして閉じる
  3. 再度メニューからウインドウ一覧画面を表示
    ※このPR適用後、OKボタンの下に隙間ができなくなる

原因は、OnInitDialog(ダイアログ初期表示イベント)内での初期化の処理順です。
修正により処理順序が適正化されるので正しく表示される、というカラクリ・・・のはず。

@beru
Copy link
Contributor Author

beru commented Jan 2, 2019

確認ありがとうございました。Mergeします。

@beru beru merged commit 1a589fa into sakura-editor:master Jan 2, 2019
@m-tmatma m-tmatma added this to the next release milestone Feb 3, 2019
HoppingTappy pushed a commit to HoppingTappy/sakura that referenced this pull request Jun 11, 2019
…ialog

ウィンドウ一覧画面を開いた後にウィンドウサイズを大きくしてから閉じて開き直すとウィンドウサイズにコントロールが追従していない問題を修正
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🐛bug🦋 ■バグ修正(Something isn't working)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants