Skip to content

fix(cli): removed alpha channel from default IOS AppIcons#10114

Merged
amrbashir merged 3 commits intotauri-apps:devfrom
pascal-audio:pr_ios_icons
Jun 26, 2024
Merged

fix(cli): removed alpha channel from default IOS AppIcons#10114
amrbashir merged 3 commits intotauri-apps:devfrom
pascal-audio:pr_ios_icons

Conversation

@lyager
Copy link
Contributor

@lyager lyager commented Jun 24, 2024

According to Apple's human interface guideline
(https://developer.apple.com/design/human-interface-guidelines/app-icons) transparent or icons with alpha channel are not allowed, and will be rejected upon upload to Apple appstore.

@lyager lyager requested a review from a team as a code owner June 24, 2024 13:18
@JonasKruckenberg
Copy link
Member

I can attest based on experience that this is true and those icons will be rejected indeed

@amrbashir amrbashir requested a review from lucasfernog June 24, 2024 16:18
@FabianLars
Copy link
Member

RemindMe! 1 day "Handle this in tauri icon"

(honestly wouldn't mind that reddit bot over here...)

FabianLars
FabianLars previously approved these changes Jun 24, 2024
Copy link
Member

@FabianLars FabianLars left a comment

Choose a reason for hiding this comment

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

Approving as a witness of Jonas' fight against Apple.

According to Apple's human interface guideline
(https://developer.apple.com/design/human-interface-guidelines/app-icons)
transparent or icons with alpha channel are not allowed, and will be rejected
upon upload to Apple appstore.
@lyager
Copy link
Contributor Author

lyager commented Jun 24, 2024

@FabianLars Commit now signed

Co-authored-by: Amr Bashir <github@amrbashir.me>
@amrbashir amrbashir changed the title bug: removed alpha channel from default IOS AppIcons fix(cli): removed alpha channel from default IOS AppIcons Jun 26, 2024
@amrbashir amrbashir merged commit 1601da5 into tauri-apps:dev Jun 26, 2024
@lyager lyager deleted the pr_ios_icons branch June 27, 2024 07:41
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.

5 participants