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

Improve notification placement #55

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

Conversation

mntone
Copy link
Contributor

@mntone mntone commented Aug 23, 2020

Changes

Refactored the entire notification codes.

  • Placement calculations have been changed to Win32 RECT-based.
  • Properties set in NotificationWindow from both SwitchWindow and PinWindow.
  • Use NativeOpacity for PinWindow.
  • PinWindow supported window placement mode (PinWindow only).
    • Added the new mode to place PinWindow outside the window. As below:
      Show on the notification outside the window
  • Changed to use DwmGetWindowAttribute to get the apparent size of the window.
  • Fixed an issue that PinWindow is out of the screen when the target window is maximized.

Settings


変更点

通知ウィンドウ周りを全体的にリファクタリングしました。

  • 配置方法の計算を Win32 RECT ベースに変更しました。
  • SwitchWindowPinWindow 両方に設定されている値を NotificationWindow で設定するようにしました。
  • PinWindowNativeOpacity の透過を使用するように変更しました。
  • PinWindow の配置変更に対応しました (PinWindow 専用)。
    • 以下のようなウィンドウの Y軸方向の外周に配置するモードを追加しました。
      Show on the notification outside the window
  • ウィンドウサイズの取得に DwmGetWindowAttribute を使用し,見かけ上のサイズを取得するように変更しました。
  • 対象のウィンドウが最大化されているとき, PinWindow が一部画面外に表示されるケースに対処しました。

Settings

@mntone mntone changed the title Improve notification Improve notification placement Aug 23, 2020
@Poopooracoocoo
Copy link

@mntone i think your additions would be accepted at ViRb3/SylphyHornEx

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.

2 participants