Skip to content
This repository has been archived by the owner on Oct 19, 2020. It is now read-only.

🆕 Remove image transparency from iOS icon-1024.png #109

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

WuglyakBolgoink
Copy link


add new command: --i1024bg= (color can be red,blue,#fff,#e2e2e2)

- - - -
add new command: --i1024bg=<color> (color can be red,blue,#fff,#e2e2e2)
@ccorcos
Copy link

ccorcos commented Nov 8, 2017

Hmm. I couldnt get this to work...

@WuglyakBolgoink
Copy link
Author

@ccorcos

image

cordova-icon --icon=../original.png --i1024bg=magenta

@ccorcos
Copy link

ccorcos commented Nov 9, 2017 via email

@BryanH
Copy link

BryanH commented Nov 27, 2017

@ccorcos It is. If not specified, it defaults to white. Look at line 17 of index.js in the PR

@ccorcos
Copy link

ccorcos commented Nov 29, 2017

Yeah, I see that works, but the icon is not showing up when I build the app...

@WuglyakBolgoink
Copy link
Author

WuglyakBolgoink commented Nov 30, 2017

@ccorcos which cordova-ios version you have?

try

cordova platform rm ios --save --fetch
cordova platform add [email protected] --save --fetch

@ccorcos
Copy link

ccorcos commented Dec 1, 2017 via email

@WuglyakBolgoink
Copy link
Author

@ccorcos in cordova-ios <v4.5.2 there are no automatically changes in *-info.plist files or in other files. You should do it manually!

See: https://cordova.apache.org/announcements/2017/10/16/ios-release.html

After I upgraded until v4.5.3 and generated icons, all was OK!

@ccorcos
Copy link

ccorcos commented Dec 4, 2017

Alright. In my config.xml:

    <engine name="ios" spec="4.5.4" />

In my package.json:

		"cordova-icon": "git://github.com/WuglyakBolgoink/cordova-icon#9ad9e153557ad99be93f9c27f3d58d40e0dc66c0",

Here's the code I'm running.

$(dirname $0)/../../../node_modules/.bin/cordova-icon \
	--config=$(dirname $0)/../config.xml \
	--icon=$(dirname $0)/../$ICON_NAME \
	--i1024bg="#fff"

When I build the application, there no icon...

@ccorcos
Copy link

ccorcos commented Dec 8, 2017

I used console.log right here and apparently, I was getting an error that wasn't bubbling up for some reason. I tried to find the bug, but I couldnt figure out why the application wasnt crashing or logging.

{ Error: Command failed: convert: unable to load module '/usr/local/Cellar/imagemagick/7.0.5-4/lib/ImageMagick//modules-Q16HDRI/coders/png.la': file not found @ error/module.c/OpenModule/1279.
convert: no decode delegate for this image format `PNG' @ error/constitute.c/ReadImage/509.
convert: no images defined `platforms/ios/Notion Local/Images.xcassets/AppIcon.appiconset/[email protected]' @ error/convert.c/ConvertImageCommand/3254.

    at ChildProcess.<anonymous> (/Users/chet/Code/notion-next/node_modules/imagemagick/imagemagick.js:88:15)
    at emitTwo (events.js:125:13)
    at ChildProcess.emit (events.js:213:7)
    at maybeClose (internal/child_process.js:927:16)
    at Socket.stream.socket.on (internal/child_process.js:348:11)
    at emitOne (events.js:115:13)
    at Socket.emit (events.js:210:7)
    at Pipe._handle.close [as _onclose] (net.js:557:12) timedOut: false, killed: false, code: 1, signal: null }

I just had to brew upgrade imagemagick and it worked! Merge it!

@ccorcos
Copy link

ccorcos commented Dec 8, 2017

@AlexDisler ^

@gleb-britecore
Copy link

@AlexDisler Any updates if we can merge it?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants