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

サイズボックスウィンドウをその都度作り直すのではなく表示・非表示を切り替えて使いまわす #695

Merged
merged 2 commits into from
Dec 16, 2018

Conversation

beru
Copy link
Contributor

@beru beru commented Dec 11, 2018

ウィンドウの分割表示中にリサイズする度にサイズボックスのウィンドウを毎回再作成していて勿体ないなと思ったので、表示非表示を切り替える方式に変更しました。

なおサイズボックスの代わりに空表示用にスタティックウィンドウの表示も行いますが、そのウィンドウのハンドル用のメンバー名は m_hwndSizeBoxPlaceholder にしました。

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.

PRありがとうございます。

みなさんの反応を見たいので判断保留にさせてください。

berryzplus
berryzplus previously approved these changes Dec 15, 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.

LGTMです。待っても反応なさそうなのでapproveしてしまいます。

直さなくても大勢に影響のない指摘をいくつか書きましたがこのままマージで問題ないと思います。

sakura_core/view/CEditView.cpp Outdated Show resolved Hide resolved
sakura_core/view/CEditView.cpp Outdated Show resolved Hide resolved
sakura_core/view/CEditView.cpp Show resolved Hide resolved
sakura_core/view/CEditView.cpp Show resolved Hide resolved
sakura_core/view/CEditView.cpp Outdated Show resolved Hide resolved
sakura_core/view/CEditView.cpp Outdated Show resolved Hide resolved
sakura_core/view/CEditView.cpp Show resolved Hide resolved
sakura_core/view/CEditView.cpp Show resolved Hide resolved
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.

対応ありがとうございます。LGTMです。

@beru
Copy link
Contributor Author

beru commented Dec 16, 2018

Review ありがとうございます。Merge します。何か問題が見つかったら別のPRで対処します。

@beru beru merged commit 0876c16 into sakura-editor:master Dec 16, 2018
@beru beru deleted the CEditView_m_hwndSizeBox branch December 16, 2018 15:26
@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
サイズボックスウィンドウをその都度作り直すのではなく表示・非表示を切り替えて使いまわす
This pull request was closed.
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

Successfully merging this pull request may close these issues.

3 participants