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

SFSymbolsの追加 #36

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

SFSymbolsの追加 #36

wants to merge 4 commits into from

Conversation

jyu0414
Copy link

@jyu0414 jyu0414 commented Oct 31, 2022

iOS / macOSで使用するSFSymbols用の形式を書き出したので追加したいです.

@hiroshi
Copy link
Member

hiroshi commented Nov 1, 2022

from https://nota.slack.com/archives/C2Z1F44PK/p1667271072820059?thread_ts=1667270505.733259&cid=C2Z1F44PK
これは svg ディレクトリに入ってたものを何らかのツールで変換したものを SFSymbols に追加したということですか?
どのくらいアプリで参照するのかわかりませんが、
なんとなく使うものだけ Gyazo(mac)/GyazoiOS でビルド時に参照して変換するとか、 Kamon にビルドの github actions 付けて release assets にしたほうがよさそうな気もするけど、いろいろめんどくさそうですね。
CC: @mountainboooy

Copy link
Member

@pastak pastak left a comment

Choose a reason for hiding this comment

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

npm install で使わないものを入れるの、現在の用法と異なる気がするので、そこは検討されて欲しいです。

PRのDescriptionにもう少し具体的な目的などが書いてあると、その辺合意しやすそうで、さらに言うとこのリポジトリに入れるなら、生成方法や利用方法などもREADMEに書いて欲しいです。

@mountainboooy
Copy link
Member

iOS上でKamonのアイコンを使用するという意図ですよね?
KamonはあくまでWebアプリ上での利用しか想定されてないので、SFSymbolsについてはAppleから提供されている純正のアイコンセットをつかっておく方が良いと思います。

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.

4 participants