Skip to content

Conversation

@Xiaoy312
Copy link
Contributor

@Xiaoy312 Xiaoy312 commented Aug 4, 2025

GitHub Issue (If applicable): closes #1404

PR Type

What kind of change does this PR introduce?

  • Bugfix
  • Refactoring (no functional changes, no api changes)

What is the current behavior?

ExtendedSplashscreen is not working on skia-mobile.

What is the new behavior?

dropped platform-specific implementations, and moved to a single shared impl.

PR Checklist

Please check if your PR fulfills the following requirements:

Other information

depends on: unoplatform/uno.resizetizer#349

@Xiaoy312 Xiaoy312 added the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Aug 4, 2025
@Xiaoy312 Xiaoy312 force-pushed the dev/xygu/20250804/unified-splashscreen branch 2 times, most recently from ee3db46 to 816be5c Compare August 4, 2025 19:52
@Xiaoy312 Xiaoy312 added the do-not-merge/breaking-change Categorizes issue or PR as related to breaking change to an existing API label Aug 15, 2025
@Xiaoy312 Xiaoy312 force-pushed the dev/xygu/20250804/unified-splashscreen branch from 816be5c to 76fc67e Compare August 15, 2025 23:50
@Xiaoy312 Xiaoy312 removed the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Aug 15, 2025
@Xiaoy312 Xiaoy312 force-pushed the dev/xygu/20250804/unified-splashscreen branch from 76fc67e to d5bcf87 Compare August 20, 2025 00:06
@Xiaoy312 Xiaoy312 removed the do-not-merge/breaking-change Categorizes issue or PR as related to breaking change to an existing API label Aug 20, 2025
@Xiaoy312 Xiaoy312 enabled auto-merge August 20, 2025 00:06
@Xiaoy312 Xiaoy312 force-pushed the dev/xygu/20250804/unified-splashscreen branch from d5bcf87 to b900150 Compare August 20, 2025 19:11
@Xiaoy312 Xiaoy312 force-pushed the dev/xygu/20250804/unified-splashscreen branch 2 times, most recently from faaba19 to 639a6bd Compare September 9, 2025 16:04
BREAKING CHANGE: unified ExtendedSplashScreen code across all platforms:
- `Platforms` and `SplashIsEnabled` properties has now been removed
- On Android, you no longer need to explicitly call `ExtendedSplashScreen.Init`
@Xiaoy312 Xiaoy312 force-pushed the dev/xygu/20250804/unified-splashscreen branch from 639a6bd to d0abd2f Compare September 11, 2025 14:59
@Xiaoy312 Xiaoy312 disabled auto-merge September 11, 2025 18:48
@Xiaoy312 Xiaoy312 added the do-not-merge/breaking-change Categorizes issue or PR as related to breaking change to an existing API label Sep 11, 2025
@Xiaoy312
Copy link
Contributor Author

DO NOT MERGE: on hold until we reach an agreement

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

do-not-merge/breaking-change Categorizes issue or PR as related to breaking change to an existing API

Projects

None yet

Development

Successfully merging this pull request may close these issues.

ExtendedSplashScreen does not show app icon when loading (Skia)

1 participant