Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

End of Year: Listening time story design #556

Merged
merged 5 commits into from
Nov 10, 2022

Conversation

ashiagr
Copy link
Contributor

@ashiagr ashiagr commented Nov 10, 2022

📘 Project: #410

Description

Add the design for the "listening time" story.

  • In order to display some podcasts on this screen, top 10 podcasts are retrieved and then display the last 3.
  • Displays dynamic background but does not add any gradient.

Testing Instructions

  1. Set END_OF_YEAR_ENABLED feature flag to true in base.gradle
  2. Run the app and make sure you're logged in to an account with a few items on Listening History
  3. When the prompt appears, tap "View My 2022"
  4. Navigate to the second story
  5. ✅ Check the design

Screenshots or Screencast

Pixel 5 Pixel 2
Tablet (Nexus 5)

Checklist

  • If this is a user-facing change, I have added an entry in CHANGELOG.md
  • I have considered whether it makes sense to add tests for my changes
  • All strings that need to be localized are in modules/services/localization/src/main/res/values/strings.xml
  • Any jetpack compose components I added or changed are covered by compose previews

I have tested any UI changes...

  • with different themes
  • with a landscape orientation
  • with the device set to have a large display and font size
  • for accessibility with TalkBack

@ashiagr ashiagr requested a review from a team as a code owner November 10, 2022 02:37
@ashiagr ashiagr mentioned this pull request Nov 10, 2022
35 tasks
@ashiagr ashiagr force-pushed the task/410-listening-time-story-design branch from 4109f1a to e8c1c1b Compare November 10, 2022 02:43
modifier = modifier
.fillMaxWidth()
.padding(top = 24.dp, start = 40.dp, end = 40.dp)
)
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can we give this 80% opacity to match the design, also should we make it slightly bolder.

Screenshot 2022-11-10 at 2 41 08 pm

Screenshot 2022-11-10 at 2 41 17 pm

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice observation! Updated in 1137927.

}
.graphicsLayer(
translationX = -translateBy.toFloat()
)
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Amazing work getting this working and multiple artworks lined up. 🏆

Base automatically changed from task/410-intro-story-design to main November 10, 2022 05:15
@ashiagr ashiagr merged commit 3331bfc into main Nov 10, 2022
@ashiagr ashiagr deleted the task/410-listening-time-story-design branch November 10, 2022 05:24
@ashiagr ashiagr added this to the 7.27 ❄️ milestone Nov 24, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants