Status Update
Comments
wi...@google.com <wi...@google.com> #2
re...@google.com <re...@google.com> #3
Adding in other folks to. This speaks to what we mentioned before for splash screen on other form factors. Was 48dp the recommendation prior to Android 12? Or is 48dp the desired size by Wear UX?
ma...@google.com <ma...@google.com> #4
Is there a reason why we can't reduce the size to 48dp when on Wear?
- Also, looking at the video
- It seems the mask reveal is not present in this context, is that right?
- The window expansion is square, but it should match the device shape.
wi...@google.com <wi...@google.com> #5
Ah wait,
Checking on code again and this should be an issue for non-phone device(wear) + API Level < S
For API level < 31, the theme "Theme.SplashScreen.IconBackground" has declare a windowbackground drawable with "compat_splash_screen", and there will specify dimensions for the icon. But that size seems was designed for phone.
The support library should provide different dimensions of theme, so it can backward compatible with other platform device with API < S.
fe...@gmail.com <fe...@gmail.com> #6
wi...@google.com <wi...@google.com> #8
I think what we need to do is to create another theme for Wear.
Hey Nick, could you share the spec for Wear like what we defined for
For developers, a workaround is that they can override the dimension Theme.SplashScreen.IconBackground
ma...@google.com <ma...@google.com> #9
Hi,
Here is a 48x48dp spec for the icon with background included, with 60 being the size for a transparent one (1.25x). I will attach a diagram to illustrate the dimensions.
N
ma...@google.com <ma...@google.com> #10
Here is the 48x48, 60x60 diagram mentioned above.
ys...@google.com <ys...@google.com> #11
I'd suggest starting with 48x48, the size is currently an external requirement for apps. If we change the size, then docs presumably should be updated.
Display the app icon in the Window background, so users see the app icon first. This improves brand recognition and is one way to show progress. Use a black window background with the app icon. The app icon must be a 48x48dp circular icon that is positioned in the center of the watch face.
le...@google.com <le...@google.com> #12
For the wear plaform side, staring on API 33, we're implementing the StartingWindowTypeAlgorithm for Wear via the WM shell integration. Let me know if there's anything specific required there to support this.
Also, it probably makes sense to provide *-watch dimens for the icon sizes as mentioned above. @maxencep for Wear UX FYI.
ma...@google.com <ma...@google.com> #14
Thanks Todd.
It does not yet align with UX expectations.
The desired behavior is similar as
-
Launch transition uses Wear task launch transition.
↪ On the video: no transition visible -
The app icon should not move (position animation) within the app view.
↪ On the video: the icon animates up on the Y axis when fading in -
The app icon can be animated via AVD following the same guidelines
↪ Not able to verify -
We use the exit animation to gracefully reveal the app once loaded
↪ On the video: the icon animates down on Y axis and the reveal/fade isn't as polished as the one on Handset/the guidelines above.
For the last one specifically, I would like the possibility to explore a different reveal that is more suited to the form factor if there are opportunities to customize it for Wear.
le...@google.com <le...@google.com> #15
Thanks for the feedback Max - sorry, I should have specified the scope of the proposed change. This appears to be addressing only the size (allowing for smaller icon sizes - i.e. compare at the clip from
ma...@google.com <ma...@google.com> #16
App icon size is correct! Thanks
ap...@google.com <ap...@google.com> #17
Branch: androidx-main
commit 0f66a201003d4f9f1d74830376744d43ede0900d
Author: wilsonshih <wilsonshih@google.com>
Date: Wed Feb 22 16:47:48 2023
Provide default dimens for wear device.
Provide default dimensions for wear device
Bug: 256678037
Test: run test on wear device.
Relnote: Provide default dimensions for wear device with 48x48dp icon
Change-Id: Ib8de8f78ea1d1dbe67cbc2d8de0388f5cfeb6c7e
A core/core-splashscreen/src/main/res/values-watch/dimens.xml
ys...@google.com <ys...@google.com> #18
Tiny nit. Can we also change the default background to black? When we use @android:style/Theme.DeviceDefault it has a black background. But we need to override the background in the splash theme when using Theme.SplashScreen as a parent.
It seems like it goes
Theme.SplashScreen > Theme.SplashScreen.Common > Base.Theme.SplashScreen.DayNight > Base.Theme.SplashScreen.Light
Or maybe I'm reading it wrong? I know it's part of the docs to override, but I got the impression on mobile, apps could omit setting the background and it would take the right one from the dark/light theme default?
dl...@gmail.com <dl...@gmail.com> #19
dl...@gmail.com <dl...@gmail.com> #20
I've just tested the solution suggested above and it works brilliantly for WearOS .. try adding the below to your dimens.xml file
<dimen name="splashscreen_icon_size_no_background" tools:override="true">288dp</dimen>
<dimen name="splashscreen_icon_mask_size_no_background" tools:override="true">410dp</dimen>
<dimen name="splashscreen_icon_mask_stroke_no_background" tools:override="true">109dp</dimen>
<dimen name="splashscreen_icon_size_with_background" tools:override="true">48dp</dimen> <!-- icon size directive -->
<dimen name="splashscreen_icon_mask_size_with_background" tools:override="true">72dp</dimen>
<dimen name="splashscreen_icon_mask_stroke_with_background" tools:override="true">64dp</dimen>
wi...@google.com <wi...@google.com> #21
Re
res/values-watch
rg...@google.com <rg...@google.com> #22
Todd, could your team prepare a CL for the background and test that it works the way you want?
le...@google.com <le...@google.com> #23
ack - we actually resolved the dark background for wear here:
Based on the work already done to adjust the sizing here:
Shall we mark this closed?
rg...@google.com <rg...@google.com> #24
Yes, that looks done... Wei Sheng, please reopen and let us know if you had something beyond that in mind.
pr...@google.com <pr...@google.com> #25
The following release(s) address this bug.It is possible this bug has only been partially addressed:
androidx.core:core-splashscreen:1.1.0-alpha02
wi...@google.com <wi...@google.com> #26
The version 1.1.0-alpha02
update the core-splashscreen library to latest verson against SDK 34, what do you mean "partially addressed"?
ys...@google.com <ys...@google.com> #27
That's a template message for these automated updates. A commit was included in a release towards this fix, but without promising it is the final change.
But you can now use both 1.1.0-alpha02 and 1.0.1 on Wear.
ys...@google.com <ys...@google.com> #28
What does the minimal correct implementation of a Splashscreen for Wear 3 look like? Is it customised theme + dimens.xml + splashscreen.xml drawable? I'm wondering why it isn't just providing an icon?
Does the androidx codebase have a sample/demo/test showing how to use this on Wear. Following the docs gets the right results, but honestly there is way more setup than I would expect.
pa...@gmail.com <pa...@gmail.com> #29
Thanks!
na...@google.com <na...@google.com> #30
The following release(s) address this bug.It is possible this bug has only been partially addressed:
androidx.core:core-splashscreen:1.2.0-alpha01
Description
Description
Currently it's not possible to specify smaller dimensions for the icon (as perhttps://developer.android.com/reference/kotlin/androidx/core/splashscreen/SplashScreen#specs ), so the recommended implementation using
androidx.splashscreen
library doesn't work on Wear OS. Wear OS splash icon must be 48x48dp.Future Wear OS platform releases that will be based on Android12+ will need to comply with our Splash Screen changes and ideally follow our recommendations for mobile as desctibed athttps://developer.android.com/develop/ui/views/launch/splash-screen/migrate
So it's important to understand if this feature request is feasible, and if not what other options do we have for Wear developers.
Versions used
Component used:
androidx.core:core-splashscreen
Version used:
1.0.0-beta02
Devices/Android versions reproduced on: Wear OS emulator API 30 (Pixel watch)
Other details
Sample project to trigger the issue: seehttps://github.com/android/wear-os-samples/pull/573 for discussion and code, it's a part of public samples for Wear OS.
A screenrecord or screenshots showing the issue (if UI related): attached video demonstrating issue, it corresponds to following theme definition: