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 notifyicon #53

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

Conversation

mntone
Copy link
Contributor

@mntone mntone commented Aug 7, 2020

Note: AcrylicBlurWindow (Grabacr07/MetroRadiance#37) is required if you build this PR.

Additional notes:

  • TextFormattingMode.Ideal is only used if the actual drawing font size is greater than 14px, otherwise TextFormattingMode.Display is used.
  • TaskTrayTestWindow is only included in the debug build. You can access it from the tasktray icon.

(以下の日本語は英語と同じ内容です)

注意: このPRをビルドするには,AcrylicBlurWindow (Grabacr07/MetroRadiance#37) が必要です。

  • 淡色テーマ用の静的アイコンを追加しました。
  • 淡色テーマ時に仮想デスクトップ番号が見づらい問題を修正しました (デスクトップの番号をタスクトレイに表示するの時の常駐アイコンがライトテーマだと見づらい #50)。
  • GDI+ の場合納得いく描画結果が得られなかったため WPF (DirectWrite) ベースのアイコン生成に書き換えました。
  • Tooltipのテキストに「デスクトップ 2/4」と言った形で表示する機能を実装しました。
  • 仮想デスクトップを切り替えずに仮想デスクトップが削除されると,通知領域のアイコンが更新されない不具合を修正しました。

補足説明:

  • TextFormattingMode.Ideal は実際の描画フォントサイズが 14px を超えている場合のみ使用し,それ以外は TextFormattingMode.Display を使用しています。
  • TaskTrayTestWindow はデバッグビルドのみ組み込まれます。タスクトレイ アイコンからアクセスできます。

In the appendix, you can see some screenshots of the various themes. I took them in non-scaled (100%) environment.

添付資料として,各種テーマ使用時のスクリーンショットを掲載します。スケーリングしていない (100%) 環境で撮影したものです。

alldisplay-100-dark-improvement

alldisplay-100-light-improvement

alldisplay-100-accent-default-improvement

alldisplay-100-accent-improvement

tasktray text


(Japanese only)

連絡事項: Grabacr07/MetroRadiance#40 を先に取り込んでいただければ,Grabacr07/MetroRadiance#37 (Grabacr07/MetroRadiance#35 も同様) の方は Grabacr07/MetroRadiance#40 の変更により適したコードで書き直します。

MetroRadiance の以下に提示した他の PR とはおそらく競合しません。

@mntone mntone force-pushed the improve-notifyicon branch from f9c2105 to 787ae41 Compare August 8, 2020 01:00
@mntone mntone force-pushed the improve-notifyicon branch from deb053e to af5baac Compare August 8, 2020 06:08
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