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

OSバージョン判定 + バルーン通知情報の余白が不適切な件について #44

Open
Asteriskx opened this issue Sep 15, 2019 · 9 comments
Assignees
Labels

Comments

@Asteriskx
Copy link
Member

Asteriskx commented Sep 15, 2019

現象

  1. Windows 8.1 にて、トースト通知がなされない
  2. バルーン通知を行ったとき、Legato NowPlayingアルバム名 となり、
    半角スペースがないので気持ちが悪い

使用OS

  • Windows 8.1 Home

対策案

  1. OSメジャーバージョン判定箇所の見直し
  2. 文字列フォーマットを修正する
@Asteriskx Asteriskx self-assigned this Sep 15, 2019
@Asteriskx Asteriskx changed the title バージョン判定がガバガバ+ バルーン通知情報の余白が一部不適切 OSバージョン判定 + バルーン通知情報の余白が不適切な件について Sep 15, 2019
@marihachi
Copy link
Contributor

2についてはどういうこと?

@Asteriskx
Copy link
Member Author

下の画像例で行くと、
NowPlaying鈴木このみ ~ ってなってると思うんだけど、
NowPlaying 鈴木このみ ってなってほしいかなと思ってて issue に追加したのよ

legato-baloon-fix

@marihachi
Copy link
Contributor

そこは本来なら改行されるはずだけど、まあそれはよくて
トースト通知が正常に発行されれば、その改行されないのは問題にならないと思う

@Asteriskx
Copy link
Member Author

おk
しかしまあ、OS 間でこうも挙動が違うとはやっかいだよねー。

@marihachi
Copy link
Contributor

トーストが普通は使われるだろうから、OS間の挙動の違いもあんまり検証されてないのかもね

@marihachi
Copy link
Contributor

あー、なるほどTitleとTextが続けて表示されちゃってるのか

@Asteriskx
Copy link
Member Author

Title と Text の余白部分はすぐ治せるけどねw

@marihachi
Copy link
Contributor

思いついた実装を一つ
メジャーバージョンとマイナーバージョンにそれぞれ重みを付けて、足し合わせた結果を比較
例えば重みの付け方が、
major * 1000 + minor * 1
だとしたら
win7(6.1)なら6001
win8(6.2)なら6002
win8.1(6.3)なら6003
win10(10.0)なら10000
という感じ
この場合、トースト通知の条件は
sum >= 6002
となりそう

@Asteriskx
Copy link
Member Author

似たような実装やってたわb
build version とかも考慮してみるかー

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants