You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@cordova.apache.org by "Jan Velecký (JIRA)" <ji...@apache.org> on 2014/07/14 20:38:06 UTC

[jira] [Commented] (CB-7132) Single default icon overridden by Cordova icon on Android

    [ https://issues.apache.org/jira/browse/CB-7132?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14061000#comment-14061000 ] 

Jan Velecký commented on CB-7132:
---------------------------------

Yes, this is changed in CB-2606 https://github.com/AxelNennker/cordova-lib/commit/60abb504d5b4001831e29c5b15c35f79e48dd524 .
If you have only default icon, icon is copied only into universal drawable folder. You should delete icons in drawable-xxx folders, then it will be fine. (And call prepare then don't broke this.)

But I think this is still bug. Cordova should handle icon changes and delete old icons, no only add (or replace If there is conflict) new icons. Same opinion have Axel Nennker (here https://github.com/apache/cordova-lib/pull/30 ) with splashreen images.

> Single default icon overridden by Cordova icon on Android 
> ----------------------------------------------------------
>
>                 Key: CB-7132
>                 URL: https://issues.apache.org/jira/browse/CB-7132
>             Project: Apache Cordova
>          Issue Type: Bug
>          Components: Android, CLI
>    Affects Versions: 3.5.0
>            Reporter: Bryan Higgins
>
> Behaviour has changed between 3.5.0-0.2.4 and 3.5.0-0.2.6 resulting in the "single default icon" [1] not being displayed on device.
> In 3.5.0-0.2.4 the icon was copied into drawable-*
> In 3.5.0-0.2.6 the icon is copied to drawable/
> The template [2] already contains default icons for drawable-*, thus the single default icon is never used.
> [1] http://cordova.apache.org/docs/en/3.5.0/config_ref_images.md.html
> [2] https://github.com/apache/cordova-android/tree/master/bin/templates/project/res



--
This message was sent by Atlassian JIRA
(v6.2#6252)