Automattic / Automattic/pocket-casts-android

Watch Now Playing and Up Next screens do not show the time

Open
#1,093 0 comments 0 reactions 0 assignees View on GitHub
[Area] Wear OS [Type] Enhancement
Dominant language
Kotlin
Stars
2.8k
Forks
308
Avg merge
1d 13h
Merged PRs (30d)
143

Description

### Description

On the watch, the Now Playing and Up Next screens do not show the time text. This was done [intentionally](https://github.com/atharv028/pocket-casts-android/blob/27bc7a4cc76a605d14ccb4dbfdefa1d32d19c7f8/wear/src/main/kotlin/au/com/shiftyjelly/pocketcasts/wear/ui/component/NowPlayingPager.kt#L60-L64) because there does not currently appear to be a good way to have the time text scroll away independently across different pages of a Pager. More details about the trouble I had with this are in [this comment](https://github.com/google/horologist/issues/1181#issuecomment-1522555773).

### Step-by-step reproduction instructions

1. Open the watch to the main watch list screen
2. ✅ Observe there is time text
3. Swipe left to the Now Playing screen
4. 🔲 Observe there is no time text
5. Swipe left to the Up Next screen
6. 🔲 Observe there is no time text

### Screenshots or screen recording

||||
| --- | --- | --- |
| ![Screenshot 2023-06-16 at 11 37 09 AM](https://github.com/Automattic/pocket-casts-android/assets/4656348/7352d590-46d0-48db-ab06-81615c719d3d) | ![Screenshot 2023-06-16 at 11 37 19 AM](https://github.com/Automattic/pocket-casts-android/assets/4656348/732c9e2c-19d7-4698-b413-f6d1870af40e) | ![Screenshot 2023-06-16 at 11 37 27 AM](https://github.com/Automattic/pocket-casts-android/assets/4656348/c23cb75e-2939-46c6-b179-4bbe2aa8b74b) |

### Did you search for existing bug reports?

- [X] I have searched for existing bug reports.

### Device, Operating system, and Pocket Casts app version

7.41-rc-2 pre-release build

Contributor guide

Open the contributing guide

Research direction

Start with wear/src/main/kotlin/au/com/shiftyjelly/pocketcasts/wear/ui/component/NowPlayingPager.kt, especially the linked section where time text was intentionally omitted. Review the related Horologist discussion before investigating the Pager behavior. Done means the watch's Now Playing and Up Next screens display the time text while preserving the expected scrolling behavior across pages.

Written by the indexing model from the issue text.

Assessment

Tech stack
kotlin
Domain
mobile
Issue type
Bug
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.