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

改善: mainWindowの画面内判定を修正 #231

Merged

Conversation

berlysia
Copy link
Member

@berlysia berlysia commented Feb 1, 2019

このpull requestが解決する内容
一部のパターンでmainWindowが画面外領域に出ることを判定できなかった問題を修正します。
たとえばマルチディスプレイかつディスプレイの付け外しを行う環境で、画面外の座標が保存されてしまい、そのまま操作できなくなる場合がありました。
#94 (review) に関係していると見えて、完全には解消していない可能性が否定できませんが、効果ありと見て変更します。

動作確認手順

  1. 何らかの方法で画面外にN AirのmainWindowを移動する。
    • タイトルバーを右クリックして「移動」を選び、矢印キーで画面外へもっていく
      • マルチディスプレイ環境でディスプレイ間に段差を作るとそこから追い出せる
  2. N Airを終了する。
  3. N Airを起動する。
  4. mainWindowがプライマリディスプレイに戻ってくる。

Copy link

@takayamaki takayamaki left a comment

Choose a reason for hiding this comment

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

設定ダイアログで画面配置を変更し、記憶されているウインドウ位置が表示領域外になった場合に、プライマリディスプレイの中央に表示される挙動を確認しました。
問題ないと思います。

↓左がプライマリディスプレイ、右がセカンダリディスプレイです
2019-02-06 15-53-13

@berlysia
Copy link
Member Author

berlysia commented Feb 6, 2019

動作確認ありがとうございます:pray:
上方向と左右方向の判定をクリアしつつ下方向にはみ出すパターンを検出できていることが確認できていそう(この条件は概要に書くべきでした)

@berlysia berlysia merged commit 5a63578 into n-air-app:n-air_development Feb 6, 2019
@berlysia berlysia deleted the fix/main-window-containment branch February 6, 2019 08:46
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