search

Legato-Dev / Legato-NowPlaying

Let's share information of songs are playing on AIMP to SNS.
Other
11 stars 2 forks source link

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

Open Asteriskx opened 5 years ago

Asteriskx commented 5 years ago

現象

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

    使用OS

    • Windows 8.1 Home

      対策案

  3. OSメジャーバージョン判定箇所の見直し
  4. 文字列フォーマットを修正する
marihachi commented 5 years ago

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

Asteriskx commented 5 years ago

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

legato-baloon-fix

marihachi commented 5 years ago

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

Asteriskx commented 5 years ago

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

marihachi commented 5 years ago

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

marihachi commented 5 years ago

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

Asteriskx commented 5 years ago

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

marihachi commented 5 years ago

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

Asteriskx commented 5 years ago

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