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

fix(template): wrong attribute to remove alpha channel #586

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

Conversation

chungxon
Copy link

This PR fixes the template after running the command to create a new config automatically:

  • Replaced the remove_alpha_channel_ios attribute with remove_alpha_ios
  • Added missing attributes adaptive_icon_foreground_inset and background_color_ios
  • Removed duplicated lines on the readme file

@@ -96,7 +96,6 @@ Shown below is the full list of attributes which you can specify within your Flu
- `true`: Override the default existing Flutter launcher icon for the platform specified
- `false`: Ignore making launcher icons for this platform
- `icon/path/here.png`: This will generate a new launcher icons for the platform with the name you specify, without removing the old default existing Flutter launcher icon.
- `image_path`: The location of the icon image file which you want to use as the app launcher icon
Copy link
Collaborator

Choose a reason for hiding this comment

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

This is still used

Please undo this removal

Copy link
Author

Choose a reason for hiding this comment

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

@MarkOSullivan94 I think this is a duplicate of the line that exists in the global section, this confuses me. Also, this one is not displayed in the IOS section. Is that right, or is ios missing this line? If not, I will revert the change. Thanks!

image

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.

2 participants