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: Add stories structure #428

Merged
merged 9 commits into from
Oct 21, 2022
Merged

Conversation

ashiagr
Copy link
Contributor

@ashiagr ashiagr commented Oct 20, 2022

📘 Project: #410

Adds the initial stories screen. No functionality is built on this PR, just the view.

iOS PR: Automattic/pocket-casts-ios#383

stories_view.mp4

To test

  1. Set END_OF_YEAR_ENABLED feature flag to true in base.gradle
  2. Run the app
  3. When the prompt appears, tap "View My 2022"
  4. ✅ Check that the stories view appear
  5. Tap close
  6. ✅ Check that the stories view is dismissed

Checklist

  • Should this change be included in the release notes? If yes, please add a line in CHANGELOG.md
  • Have you tested in landscape? - iOS restricts the screen to portrait mode, this will be taken up in a future PR
  • Have you tested accessibility with TalkBack?
  • Have you tested in different themes?
  • Does the change work with a large display font?
  • Are all the strings localized?
  • Could you have written any new tests?
  • Did you include Compose previews with any components?

@ashiagr ashiagr requested a review from a team as a code owner October 20, 2022 12:40
Comment on lines +22 to +28
fun SegmentedProgressIndicator(
@FloatRange(from = 0.0, to = 1.0) progress: Float,
modifier: Modifier = Modifier,
color: Color = Color.White,
backgroundColor: Color = color.copy(alpha = IndicatorBackgroundOpacity),
numberOfSegments: Int,
) {
Copy link
Contributor Author

@ashiagr ashiagr Oct 20, 2022

Choose a reason for hiding this comment

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

Most of the logic is taken from Compose internal implementation of LinearProgressIndicator and this answer.

It doesn't support RTL animation right now.

@ashiagr ashiagr mentioned this pull request Oct 20, 2022
35 tasks
Copy link
Contributor

@mchowning mchowning left a comment

Choose a reason for hiding this comment

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

This is really nice Ashita. Great job!

@ashiagr ashiagr enabled auto-merge October 21, 2022 07:34
@ashiagr ashiagr merged commit f4114e8 into main Oct 21, 2022
@ashiagr ashiagr deleted the task/410-stories-structure branch October 21, 2022 07:48
@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