[@capacitor/splash-screen] Android 8–11: failed Android 12 API attempt leaves a pre-draw hold, so nothing is drawn for launchShowDuration

未关闭 适合新手
#2,635 0 条评论 0 个 reaction 已指派 0 人 在 GitHub 查看

维护者通常 1 天内回复

还没有人认领这个 Issue。

评估

难度
2/5
预计耗时
1-3 小时
新手友好度
88/100
Issue 类型
缺陷
描述清晰度
描述清楚
活跃度
活跃
技术栈
android, java
领域
frontend, mobile

调研方向

找到 Android 启动画面插件的代码,其中 showOnLaunch() 会调用 showWithAndroid12API(),并回退到旧版 show() 路径。Issue 提供了一个补丁,根据 SDK 版本对 Android 12 API 调用进行保护。将该改动与现有行为进行比较,并验证 Android 8–11 使用旧版启动画面,而 Android 12+ 保持当前路径。

由索引模型根据 Issue 内容生成。

描述

platform: android

🤖 Automated review by Claude — posted via @ibrahimkhatib365's account; not written by them personally.

Bug Report

Plugin(s)

@capacitor/splash-screen 8.0.2. The same code is in every version from 4.x through 8.0.3 nightly, 9.0.0-alpha.2 and the current 9.0 dev builds.

Capacitor Version
Latest Dependencies:

  @capacitor/cli: 8.6.0
  @capacitor/core: 8.6.0
  @capacitor/android: 8.6.0
  @capacitor/ios: 8.6.0

Installed Dependencies:

  @capacitor/cli: 8.5.2
  @capacitor/core: 8.5.2
  @capacitor/android: 8.5.2
  @capacitor/ios: 8.5.2

[success] Android looking great! 👌
Platform(s)

Android 8–11 (API 26–30). Android 12+ is not affected.

Current Behavior

On Android 8–11, showOnLaunch() calls showWithAndroid12API() first. On those versions the attempt does the following:

  1. installSplashScreen() succeeds. The compat library supports API 23+.
  2. setKeepOnScreenCondition(() -> isVisible || isHiding) succeeds. Below API 31, the compat SplashScreen.Impl adds an OnPreDrawListener to android.R.id.content right away.
  3. With the default launchFadeOutDuration (200), setOnExitAnimationListener() then throws, because building the compat SplashScreenViewProvider needs Theme.SplashScreen attributes that a normal launch theme doesn't have.

The catch logs Android 12 Splash API failed... using previous method. and falls back to show(..., true). However, the compat pre-draw listener from step 2 is still registered, and the legacy path then sets isVisible = true. As a result, the activity's content is held back from drawing for the entire launchShowDuration. The legacy splash window shares the activity's window token, so it doesn't appear while the activity hasn't drawn either.

The visible result depends on timing:

  • On a cold launch, the first draw usually happens before isVisible flips, so the splash shows normally.
  • When the activity is recreated in an existing process (for example, after closing it with Back and reopening it from the launcher), the hold wins. Nothing of the app is drawn for launchShowDuration, and then the app appears at once.

On an Android 9 emulator with launchShowDuration: 3000, the reopen after Back logs Displayed .MainActivity: +3s426ms, and the next line in the log is SplashScreen was automatically hidden after the launch timeout. With a translucent launch theme, the launcher stays on screen for the whole 3.4 s. With an opaque launch theme, the window background shows instead of the plugin's splash.

The 9.x change to the default launchFadeOutDuration (0, #2526) makes this worse below Android 12. Step 3 no longer throws, so the plugin never reaches the legacy path. It only holds the first frame for launchShowDuration behind the theme's window. With launchFadeOutDuration: 0 on 8.0.2, Android 9 showed no splash at all on either a cold launch (+4.4 s) or a reopen (+3.2 s), only the launcher until the app appeared.

Expected Behavior

On Android 8–11, the legacy splash shows right away and the activity isn't held. That is what happens on a cold launch today.

Code Reproduction

No sample repository yet. The steps on a new app:

  1. npm init @capacitor/app, add @capacitor/splash-screen, and set SplashScreen.launchShowDuration to 3000 in capacitor.config. Keep the other defaults.
  2. Optional, to make the symptom obvious: add <item name="android:windowIsTranslucent">true</item> to AppTheme.NoActionBarLaunch.
  3. Run on an API 26–30 emulator, for example Android 9.
  4. Launch the app and wait for it to finish loading, press Back, then tap the launcher icon again.
  5. The app (and the splash) are not drawn for about 3 s. adb logcat shows Android 12 Splash API failed... and an ActivityManager: Displayed time of about launchShowDuration.
Other Technical Details
  • androidx.core:core-splashscreen 1.2.0 (the plugin default)
  • Reproduced on an Android 9 (API 28) emulator
  • minSdkVersion 26
Additional Context

A guard that only uses the Android 12 API where it applies fixes this, and we now ship it as a patch:

         try {
-            showWithAndroid12API(activity, settings);
-            return;
+            if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.S) {
+                showWithAndroid12API(activity, settings);
+                return;
+            }
         } catch (Exception e) {

With this guard on Android 9, the reopen after Back shows the legacy splash immediately (Displayed +182ms). The cold launch is unchanged, and Android 12+ runs the same code as before.

Another approach is to remove the pre-draw listener in the catch. However, the compat library doesn't expose it, so skipping the attempt below API 31 seems simpler. This is related to, but separate from, #2627 and #2548, which are about how hide() removes the plugin's own pre-draw listener.

主要语言
Java
星标
688
派生
691
平均合并
1 天 5 小时
30 天内合并 PR
5

环境准备

  • 没有 Dockerfile 或 Docker Compose 文件
  • 没有 Pull Request 模板
  • 阅读贡献指南

从这里开始

  1. 先读完整个 Issue,再读项目的贡献指南。
  2. 在 Issue 下留言说明你要接手 —— 这能避免两个人做同样的事。
  3. Fork 仓库,在一个分支上完成修改。
  4. 提交 Pull Request,并在描述里引用这个 Issue 编号。

ionic-team/capacitor-plugins 的其他 Issue

查看 ionic-team/capacitor-plugins 的全部 Issue

相似的 Issue

更多 Java Issue

把新 issue 发到你的邮箱

精选适合新手参与的 GitHub issue 摘要。