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

Fix ChromeWindow bug. #46

Open
wants to merge 7 commits into
base: develop
Choose a base branch
from

Conversation

mntone
Copy link
Contributor

@mntone mntone commented Aug 25, 2020

Changes

  • Fixed an issue that the position of ChromeWindow is misplaced in the following three cases.
    • The main window is initialized by different DPI from System DPI.
    • The DPI is changed during resizing with a boundary.
    • The main window returns from a minimized state.
    • The OS font size setting has been changed.
  • Changed to show the contents of both left and right ChromeWindow from the edge of the window's border.
    No-patched  Patched
    metroradiance-fix-leftrightchrome-before    metroradiance-fix-leftrightchrome-after
  • Fixed an issue that ChromeWindow doesn't light up in case of an external window.
  • Fixed an issue that ChromeWindow's position isn't correct in case of external window.
  • Added a GUI for glowing external windows.
    new GUI

変更点

  • 以下の4ケースにおいて,ChromeWindowの位置がずれる不具合の修正。
    • System DPI 以外の DPI で初期化された場合
    • 境界線を使ってリサイズ中に DPI が変更された場合
    • 最小化状態から復帰する場合
    • OSの文字サイズ設定が変更された場合
  • 左右の ChromeWindow のコンテンツ表示を境界線の端から表示できるように変更しました。
    変更前    変更後
    metroradiance-fix-leftrightchrome-before    metroradiance-fix-leftrightchrome-after
  • 外部ウィンドウの場合,ChromeWindow が光らない問題の修正。
  • 外部ウィンドウの場合,ChromeWindow の位置がずれる問題の修正。
  • 外部ウィンドウを光らせるための GUI を追加しました。
    new GUI

すみません,#44 のデバッグが不十分でした。

補足 (Japanese only)

手元の Visual Studio 2015 でビルドしたものだと,外部ウィンドウを光らせる機能が正常に動作しない (外部ウィンドウからのイベントが飛んでこない) みたいです。
Visual Studio 2019 に移行するための PR を準備中なのですが,そちらの .NET Framework 4.5.2 の場合は動くのですが😢
原因がよくわかっていないため,外部ウィンドウ関連のみ Visual Studio 2019 でビルドしたものでテストしています。

また,.NET Core 3.1 では C++/CLI 関連のロードに失敗するようで,正常に動作しませんでした。

@mntone mntone changed the title Fix ChromeWindow per-monitor DPI bug. Fix ChromeWindow bug. Aug 25, 2020
nishy2000 added a commit to nishy2000/MetroRadiance.Fork that referenced this pull request Sep 15, 2020
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.

1 participant