You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@cordova.apache.org by GitBox <gi...@apache.org> on 2022/06/14 09:14:37 UTC

[GitHub] [cordova-android] erisu opened a new pull request, #1441: feat: android 12 splash screen

erisu opened a new pull request, #1441:
URL: https://github.com/apache/cordova-android/pull/1441

   ### Motivation and Context
   <!-- Why is this change required? What problem does it solve? -->
   <!-- If it fixes an open issue, please link to the issue here. -->
   
   Support only Android 12 Splash Screen
   
   ### Description
   <!-- Describe your changes in detail -->
   
   **Old Splash Screen**
   
   * Drop old copy resources for previous previous splash screen plugin.
   * Remove old resource files
   
   **New Splash Screen**
   
   * Created new template resources
   * Added compat library to support older devices
   * Added new `preference`
     * `AndroidPostSplashScreenTheme`
       * Considered as advance feature and use at own risk. Unsupported feature.
     * `AndroidWindowSplashScreenAnimationDuration`
       * Manages the duration of the animation, not the delay of the splash screen.
     * `AndroidWindowSplashScreenAnimatedIcon`
       * This contains the icon that is display on the splash screen.
       * Icon image can be either `xml` Android Drawable, or `png`.
       * Follow the design specs from Android: https://developer.android.com/guide/topics/ui/splash-screen#splash_screen_dimensions
     * `AndroidWindowSplashScreenBackground`
       * Background color of the splash screen. Default is white (`#FFFFFF`)
     * `AndroidWindowSplashScreenBrandingImage`
       * Adds an image to the bottom of the splash screen.
     * `AndroidWindowSplashScreenIconBackgroundColor`
   
   
   ### Testing
   <!-- Please describe in detail how you tested your changes. -->
   
   - `npm t`
   - `cordova build android`
   - `cordova run android`
   
   | API 31 | API 23 |
   | -- | -- |
   | ![Screenshot_20220614_175352](https://user-images.githubusercontent.com/1029107/173540350-a9aeb72b-a58a-460a-aa4c-9482883b0f1e.png) | ![Screenshot_1655196639](https://user-images.githubusercontent.com/1029107/173540290-ec869b2c-ae03-4bd1-a5f5-6ccfa83a2441.png) |
   
   Note: 
   * Some simulators do not display the splash screen when launch from Android Studio. This appears to be a bug from AS. Launching the app from within the simulator will show the splash screen.
   * The default splash screen is an XML Android Drawable. It has a resolution of 512x512. It is padded and the inner icon size is 384x384. It is also scalled by 50% and pivotX/Y of 256.
   
   ### Checklist
   
   - [x] I've run the tests to see all new and existing tests pass
   - [ ] I added automated test coverage as appropriate for this change
   - [ ] Commit is prefixed with `(platform)` if this change only applies to one platform (e.g. `(android)`)
   - [ ] If this Pull Request resolves an issue, I linked to the issue in the text above (and used the correct [keyword to close issues using keywords](https://help.github.com/articles/closing-issues-using-keywords/))
   - [ ] I've updated the documentation if necessary
   


-- 
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.

To unsubscribe, e-mail: issues-unsubscribe@cordova.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


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


[GitHub] [cordova-android] erisu merged pull request #1441: feat!: android 12 splash screen

Posted by GitBox <gi...@apache.org>.
erisu merged PR #1441:
URL: https://github.com/apache/cordova-android/pull/1441


-- 
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.

To unsubscribe, e-mail: issues-unsubscribe@cordova.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


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


[GitHub] [cordova-android] NelsonChad commented on pull request #1441: feat!: android 12 splash screen

Posted by GitBox <gi...@apache.org>.
NelsonChad commented on PR #1441:
URL: https://github.com/apache/cordova-android/pull/1441#issuecomment-1211742806

   Hi ,when i run cordova platform add android@11.0.0 I'm getting this error in prepare.js file
   
   ```
   TypeError: Cannot read properties of null (reading 'find')
   at E:\DEV\IONIC\donateApp_cord\node_modules\cordova-android\lib\prepare.js:387:49
   at Array.forEach (<anonymous>)
   at updateProjectSplashScreen (E:\DEV\IONIC\donateApp_cord\node_modules\cordova-android\lib\prepare.js:384:7)
   at updateProjectAccordingTo (E:\DEV\IONIC\donateApp_cord\node_modules\cordova-android\lib\prepare.js:269:5)
   at E:\DEV\IONIC\donateApp_cord\node_modules\cordova-android\lib\prepare.js:67:21
   at processTicksAndRejections (node:internal/process/task_queues:96:5)
   at async Promise.all (index 0)
   [ERROR] An error occurred while running subprocess cordova
   ```.
   
   I've configurated de config.xml as recommended, can you help me please?


-- 
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.

To unsubscribe, e-mail: issues-unsubscribe@cordova.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


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


[GitHub] [cordova-android] erisu commented on pull request #1441: feat: android 12 splash screen

Posted by GitBox <gi...@apache.org>.
erisu commented on PR #1441:
URL: https://github.com/apache/cordova-android/pull/1441#issuecomment-1168110534

   > Looks good, but if I open the themes.xml in Android Studio it shows "errors" because it doesn't understand the theme properties. The templates's app/build.gradle should include `implementation "androidx.core:core-splashscreen:${cordovaConfig.ANDROIDX_CORE_SPLASHSCREEN_VERSION}"` so it understands the properties.
   
   I added the dependency back the app as well.
   
   > Also, current default delay is 3000 and it's being changed to -1, we have to remember to document the "breaking" change
   
   I am writting the splash screen docs as well. Since the plugin does not support iOS or Android, the repo should not keep the docs anyways cause it will lead to confusion on what support the plugin has. Those docs are being removed from the plugin repo.


-- 
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.

To unsubscribe, e-mail: issues-unsubscribe@cordova.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


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


[GitHub] [cordova-android] NiklasMerz commented on pull request #1441: feat: android 12 splash screen

Posted by GitBox <gi...@apache.org>.
NiklasMerz commented on PR #1441:
URL: https://github.com/apache/cordova-android/pull/1441#issuecomment-1166245529

   
   > It seems that branding is actually not supported below API 31 and there is an issue ticket open about it. https://issuetracker.google.com/issues/194301890
   > 
   > 
   > 
   > Maybe I can remove the branding code since it is not working and dont know if it will be supported.
   
   Or we just leave it in for future versions and put a hint in the docs?


-- 
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.

To unsubscribe, e-mail: issues-unsubscribe@cordova.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


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


[GitHub] [cordova-android] codecov-commenter commented on pull request #1441: feat: android 12 splash screen

Posted by GitBox <gi...@apache.org>.
codecov-commenter commented on PR #1441:
URL: https://github.com/apache/cordova-android/pull/1441#issuecomment-1165168570

   # [Codecov](https://codecov.io/gh/apache/cordova-android/pull/1441?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation) Report
   > Merging [#1441](https://codecov.io/gh/apache/cordova-android/pull/1441?src=pr&el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation) (73351c7) into [master](https://codecov.io/gh/apache/cordova-android/commit/68a9a3181aae6e39d6812222087456358b8b377e?el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation) (68a9a31) will **decrease** coverage by `3.10%`.
   > The diff coverage is `15.46%`.
   
   ```diff
   @@            Coverage Diff             @@
   ##           master    #1441      +/-   ##
   ==========================================
   - Coverage   75.96%   72.86%   -3.11%     
   ==========================================
     Files          21       21              
     Lines        1677     1732      +55     
   ==========================================
   - Hits         1274     1262      -12     
   - Misses        403      470      +67     
   ```
   
   
   | [Impacted Files](https://codecov.io/gh/apache/cordova-android/pull/1441?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation) | Coverage Δ | |
   |---|---|---|
   | [lib/Api.js](https://codecov.io/gh/apache/cordova-android/pull/1441/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation#diff-bGliL0FwaS5qcw==) | `50.61% <ø> (ø)` | |
   | [lib/prepare.js](https://codecov.io/gh/apache/cordova-android/pull/1441/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation#diff-bGliL3ByZXBhcmUuanM=) | `48.31% <15.46%> (-11.69%)` | :arrow_down: |
   
   ------
   
   [Continue to review full report at Codecov](https://codecov.io/gh/apache/cordova-android/pull/1441?src=pr&el=continue&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation).
   > **Legend** - [Click here to learn more](https://docs.codecov.io/docs/codecov-delta?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation)
   > `Δ = absolute <relative> (impact)`, `ø = not affected`, `? = missing data`
   > Powered by [Codecov](https://codecov.io/gh/apache/cordova-android/pull/1441?src=pr&el=footer&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation). Last update [68a9a31...73351c7](https://codecov.io/gh/apache/cordova-android/pull/1441?src=pr&el=lastupdated&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation). Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation).
   


-- 
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.

To unsubscribe, e-mail: issues-unsubscribe@cordova.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


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


[GitHub] [cordova-android] ATaysikuu commented on pull request #1441: feat!: android 12 splash screen

Posted by GitBox <gi...@apache.org>.
ATaysikuu commented on PR #1441:
URL: https://github.com/apache/cordova-android/pull/1441#issuecomment-1227341017

   Hey, same issue here trying to upgrade a cordova 10 based project to cordova 11.
   
   ```
   Cannot read property 'find' of null
   TypeError: Cannot read property 'find' of null
       at C:\Jenkins\workspace\REDACTEDPROJECTNAME\node_modules\cordova-android\lib\prepare.js:387:49
       at Array.forEach (<anonymous>)
       at updateProjectSplashScreen (C:\Jenkins\workspace\REDACTEDPROJECTNAME\node_modules\cordova-android\lib\prepare.js:384:7)
       at updateProjectAccordingTo (C:\Jenkins\workspace\REDACTEDPROJECTNAME\node_modules\cordova-android\lib\prepare.js:269:5)
       at C:\Jenkins\workspace\REDACTEDPROJECTNAME\node_modules\cordova-android\lib\prepare.js:67:21
       at processTicksAndRejections (internal/process/task_queues.js:95:5)
       at async Promise.all (index 0)
   [ERROR] An error occurred while running subprocess cordova.
           
           cordova platform add android@11 --verbose --save exited with exit code 1.
           
           Re-running this command with the --verbose flag may provide more information.
   ```


-- 
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.

To unsubscribe, e-mail: issues-unsubscribe@cordova.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


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


[GitHub] [cordova-android] erisu commented on pull request #1441: feat: android 12 splash screen

Posted by GitBox <gi...@apache.org>.
erisu commented on PR #1441:
URL: https://github.com/apache/cordova-android/pull/1441#issuecomment-1165801362

   > I have some trouble with the branding image and think the documentation PR will be very important for this.
   
   It seems that branding is actually not supported below API 31 and there is an issue ticket open about it. https://issuetracker.google.com/issues/194301890
   


-- 
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.

To unsubscribe, e-mail: issues-unsubscribe@cordova.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


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