-
Notifications
You must be signed in to change notification settings - Fork 406
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
iOS Dark and Tinted icons are not placed in the correct directory #596
Comments
@vlazdra can you please provide me with a link to the documentation which states this? I tested the v0.14.1 dark icon and tinted icon implementation tonight with no issues thankfully |
@MarkOSullivan94 I haven't had a chance to review the documentation thoroughly, but when I open the Xcode preview of the icon catalog, I notice that for the icon you've set up, there are additional versions on the right side. Assuming the UI in Xcode serves as the guide, I believe that if I'm creating different versions of the same icon, they should be placed within the same catalog that already contains all existing icon sizes and dimensions, including the newly added dark and tinted icons. Since these are variations of the same launcher icon, they should reside in the same launcher icon catalog. Currently, my launcher icon is named
However, the current version of the library is creating an additional appiconset for the dark and tinted versions for each flavor, which seems unnecessary. Additionally, the When I run the command to generate all the flavors in my project, it unnecessarily creates additional catalogs that aren't properly linked with the main icon set. Here's a screenshot illustrating this issue: On the other hand, with the changes I've introduced in this PR, the Dark and Tinted icons are generated inside my main app icon set folder, ensuring they are properly linked and displayed in Xcode: Note: The |
@KevinHaendel I know you worked on this feature, what's your thoughts? |
ℹ️ Info
0.14.1
💬 Description
When running the command it creates an additional folder for the Dark and for the Tinted icons which is not how it's supposed to be. The Dark and Tinted icons should be placed in the already existing or the main AppIcon catalog (taking into account the flavor part if present)
📜 Pubspec.yaml
The text was updated successfully, but these errors were encountered: