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

change: menubar icon / TISIconIsTemplate support #33

Merged
merged 2 commits into from
Aug 3, 2024

Conversation

hazi
Copy link
Contributor

@hazi hazi commented Aug 3, 2024

白黒のアイコンから、OS同様に背景の色に合わせて色が変わる TISIconIsTemplate に対応するアイコンに変更しました。

image

次の画像はPhotoshopで作成したプレビューです。

image

念の為、元データのPSDこちらに置いておきます。

main.psd.zip

@ensan-hcl
Copy link
Owner

@hazi ありがとうございます!!めっちゃ良いです。

こちら、原理的にはmainw.tiffを追加で用意して、<key>tsInputMethodAlternateIconFileKey</key>として追加し、TISIconIsTemplateを設定すれば良い感じでしょうか?英字入力モードのアイコンも同様にしたく!

@@ -74,7 +78,7 @@
551398DA2BDD39B700F1DB82 /* StringConfigItem.swift */ = {isa = PBXFileReference; lastKnownFileType = sourcecode.swift; path = StringConfigItem.swift; sourceTree = "<group>"; };
551398DC2BDD3ABF00F1DB82 /* CustomCodableConfigItem.swift */ = {isa = PBXFileReference; lastKnownFileType = sourcecode.swift; path = CustomCodableConfigItem.swift; sourceTree = "<group>"; };
5523B1A32BF116CB0051DAA8 /* IntConfigItem.swift */ = {isa = PBXFileReference; lastKnownFileType = sourcecode.swift; path = IntConfigItem.swift; sourceTree = "<group>"; };
5557C42B2BF3C9130048F976 /* zenz-v2-Q5_K_M.gguf */ = {isa = PBXFileReference; lastKnownFileType = file; name = "zenz-v2-Q5_K_M.gguf"; path = "zenz-v2-gguf/zenz-v2-Q5_K_M.gguf"; sourceTree = "<group>"; };
5557C42B2BF3C9130048F976 /* zenz-v2-Q5_K_M.gguf */ = {isa = PBXFileReference; lastKnownFileType = text; name = "zenz-v2-Q5_K_M.gguf"; path = "zenz-v2-gguf/zenz-v2-Q5_K_M.gguf"; sourceTree = "<group>"; };
Copy link
Owner

Choose a reason for hiding this comment

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

この差分はおそらく不要なので、ここだけrevertお願いできますか?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

おっと失礼しました!
対応します。

@ensan-hcl ensan-hcl added the enhancement New feature or request label Aug 3, 2024
@hazi
Copy link
Contributor Author

hazi commented Aug 3, 2024

@ensan-hcl

こちら、原理的にはmainw.tiffを追加で用意して、tsInputMethodAlternateIconFileKeyとして追加し、TISIconIsTemplateを設定すれば良い感じでしょうか?英字入力モードのアイコンも同様にしたく!

はい!その通りです。
アイコンは他のものも同じにしても良ければ、追加ではなく置換えでもいいかもしれません。
影響範囲がわからなかったので今回追加にしております。

必要であれば置き換えます。

@ensan-hcl
Copy link
Owner

@hazi
なるほど!一旦全て置き換えてしまった方がシンプルそうなので、そちらでお願いしても良いですか?

@hazi
Copy link
Contributor Author

hazi commented Aug 3, 2024

@ensan-hcl 英語版のアイコンも合わせて対応しましょうか?
ディテールにこだわりがなければ今のものをベースに真似て作ります。
もし高解像度の元データとかあればお送りいただけると。

@ensan-hcl
Copy link
Owner

お願いしてもよろしいでしょうか?1024x1024サイズの透過pngがこちらになります。
a

@hazi
Copy link
Contributor Author

hazi commented Aug 3, 2024

image

@hazi hazi requested a review from ensan-hcl August 3, 2024 15:28
Copy link
Owner

@ensan-hcl ensan-hcl left a comment

Choose a reason for hiding this comment

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

ありがとうございます!!Mergeします!

@ensan-hcl ensan-hcl merged commit 2bf499b into ensan-hcl:main Aug 3, 2024
@hazi hazi deleted the TISIconIsTemplate branch August 3, 2024 15:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants