cordova-icon icon indicating copy to clipboard operation
cordova-icon copied to clipboard

🆕 Remove image transparency from iOS icon-1024.png

Open WuglyakBolgoink opened this issue 8 years ago • 12 comments


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

WuglyakBolgoink avatar Nov 08 '17 15:11 WuglyakBolgoink

Hmm. I couldnt get this to work...

ccorcos avatar Nov 08 '17 20:11 ccorcos

@ccorcos

image

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

WuglyakBolgoink avatar Nov 09 '17 08:11 WuglyakBolgoink

Hmm, i thought --i1024bg is optional...

On Thu, 9 Nov 2017 at 00:27 Wuglyak Bolgoink [email protected] wrote:

@ccorcos https://github.com/ccorcos

[image: image] https://user-images.githubusercontent.com/2271337/32595412-2bb68186-c530-11e7-8b60-be0b579092e0.png

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

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/AlexDisler/cordova-icon/pull/109#issuecomment-343081745, or mute the thread https://github.com/notifications/unsubscribe-auth/ABth3xZymk6DCqPiaKlJ8zGY1UPQLntzks5s0reDgaJpZM4QWhd6 .

ccorcos avatar Nov 09 '17 18:11 ccorcos

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

BryanH avatar Nov 27 '17 20:11 BryanH

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

ccorcos avatar Nov 29 '17 22:11 ccorcos

@ccorcos which cordova-ios version you have?

try

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

WuglyakBolgoink avatar Nov 30 '17 08:11 WuglyakBolgoink

4.4.0

On Thu, 30 Nov 2017 at 00:20 Wuglyak Bolgoink [email protected] wrote:

@ccorcos https://github.com/ccorcos which cordova-ios version you have?

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/AlexDisler/cordova-icon/pull/109#issuecomment-348115112, or mute the thread https://github.com/notifications/unsubscribe-auth/ABth3xECyge00X4yab-Nf4xgkS0-NAf8ks5s7mUxgaJpZM4QWhd6 .

ccorcos avatar Dec 01 '17 00:12 ccorcos

@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!

WuglyakBolgoink avatar Dec 02 '17 15:12 WuglyakBolgoink

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 avatar Dec 04 '17 19:12 ccorcos

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 avatar Dec 08 '17 00:12 ccorcos

@AlexDisler ^

ccorcos avatar Dec 08 '17 00:12 ccorcos

@AlexDisler Any updates if we can merge it?

gleb-britecore avatar Feb 08 '18 17:02 gleb-britecore