[GitHub] [cordova-android] dominic-simplan opened a new issue #690: Fallback for adaptive icon overwrites adaptive icon

[GitBox <gi...@apache.org>]

dominic-simplan opened a new issue #690: Fallback for adaptive icon overwrites adaptive icon
URL: https://github.com/apache/cordova-android/issues/690
 
 
   # Bug Report
   
   ## Problem
   
   ### What is expected to happen?
   The [documentation](https://cordova.apache.org/docs/en/dev/config_ref/images.html#adaptive-icon-with-images) says:
   
   > Note: In this example, the foreground image will also be used as the fallback icon for Android devices that do not support the adaptive icons. The fallback icon can be overridden by setting the src attribute.
   
   Therefore, I have specified my icons with a fallback as follow:
   ```
   <platform name="android">
           <resource-file src="res/icons/android/colors.xml" target="/app/src/main/res/values/colors.xml" />
           <icon density="ldpi" src="res/icons/android/ldpi-legacy.png" />
           <icon density="mdpi" background="@color/background" foreground="res/icons/android/mdpi-foreground.png" src="res/icons/android/mdpi-legacy.png" />
           <icon density="hdpi" background="@color/background" foreground="res/icons/android/hdpi-foreground.png" src="res/icons/android/hdpi-legacy.png" />
           <icon density="xhdpi" background="@color/background" foreground="res/icons/android/xhdpi-foreground.png" src="res/icons/android/xhdpi-legacy.png" />
   ...
   </platform>
   
   ```
   
   I would expect that the *-legacy.png Icons are only used on devices which do not support adaptive icons (Android < 8).
   
   ### What does actually happen?
   The legacy icons are displayed even on devices which support adaptive icons (Android 8.1.0 device)
   
   
   ## Information
   The documentation also says:
   
   > To use the adaptive icons the background and foreground attributes must be defined in place of the src attribute. The src attribute is not used for adaptive icons.
   
   Which is somewhat contrary to the other statement. So I am not really sure how it is actually supposed to behave.
   
   Btw., the latest documentation with the description for the adaptive icon does not seem to be released yet (At least it says it is under development). Are you aware of that?
   
   
   ### Environment, Platform, Device
   BQ Aquaris Pro (Android 8.1.0)
   
   
   ### Version information
   Cordova CLI 8.1.2
   Cordova Android 8.0.0
   
   
   ## Checklist
   - [x] I searched for existing GitHub issues
   - [x] I updated all Cordova tooling to most recent version
   - [x] I included all the necessary information above
   

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services

---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@cordova.apache.org
For additional commands, e-mail: commits-help@cordova.apache.org