Page MenuHomePhabricator

[Compose] Convert WatchlistExpiryDialog to use Compose
Closed, ResolvedPublic5 Estimated Story Points

Description

Requirements
  1. Convert the WatchlistExpiryDialog to use Compose.
  2. Make sure the ExtendedBottomSheetDialogFragment is able to be used by a non-Compose method.
  3. Make sure the Compose layout can work properly and correctly with the existing ViewModel.
  4. Share screenshots for the before vs after the conversion.

Event Timeline

cooltey set the point value for this task to 5.Mar 20 2025, 5:37 PM

Hi @SChekfa-WMF
This is ready for design review.

non_compose.png (1×1 px, 820 KB)
compose_1.png (1×1 px, 820 KB)
Non Compose VersionCompose Version

apologies for missing this, it looks good to me! thank you @WRai-WMF.

ABorbaWMF subscribed.

Failed on 2.7.50542-r-2025-07-28

Tested on Pixel 6 on Android 15 and OnePlus 8 on Android 13. In the dark modes, the watch expiration dialog remains in light theme.

Screenshot_20250729-121543.png (1×2 px, 122 KB)

Screenshot_20250729-121509.png (2×1 px, 881 KB)

Looks good on 2.7.50543-r-2025-07-31

Tested on Pixel 6 on Android 15 and OnePlus 8 on Android 13. Tested watch and unwatch