-
Notifications
You must be signed in to change notification settings - Fork 64
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 Icon Design Organization #52
Comments
Hey @FIGBERT, thanks for thinking about this! I like the idea but I'm also considering removing Instead of baking rounded corners and borders into the icon images, I'm thinking of having the app do these at runtime when displaying the icons. Furthermore, since the 1x and 2x sizes of the icons are typically downsized versions of the 3x icons, I'm also considering having these generated by a script included in the project. This script could also generate the This would allow the following structure:
My hope is that this would make the process of adding icons to the app smoother. What are your thoughts on this? |
I think that generating the icons with a script is really good idea, and having thought about it for a moment shouldn't be too difficult to implement. Each icon would be given in the 3x size and placed in the Moving the rounded corners and borders into code rather than the images also seems sensible to me, though I'm more familiar with SwiftUI than UIKit and may need some guidance and some additional Googling. Regardless, I think it's a really good proposal and would love to help see it through. |
I believe this can be closed with the completion of #60. Update: realized I can close myself lol |
At the moment, custom issuer icons are stored in
IssuerIcons.sketch
. This is a bit of a problem, as different pull requests that modify the file will overwrite it each other (git
doesn't track changes in binary files like.sketch
). This may have already happened, though I'm not sure, as several of the icons inAssets.xcassets
are missing fromIssuerIcons.sketch
.I would propose a new structure, that replaces the
IssuerIcons.sketch
file with a directory of icons in.sketch
format. These files would be editable in Sketch, like before, but would be safe from accidentally deletion or modification. Additionally, these files could be stored withgit-lfs
to prevent the repo from accruing bloat.The current directory structure is something along the lines of:
Under the proposed changes, it would be structured as such:
I would be more than happy to contribute this myself, but figured I'd check in, provide some reasoning, and allow for some feedback before jumping straight in.
The text was updated successfully, but these errors were encountered: