Skip to content
This repository has been archived by the owner on Nov 1, 2022. It is now read-only.

Closes #2392 - Add ability for progress bar to be displayed on top of… #2393

Closed
wants to merge 1 commit into from
Closed

Closes #2392 - Add ability for progress bar to be displayed on top of… #2393

wants to merge 1 commit into from

Conversation

ekager
Copy link
Contributor

@ekager ekager commented Mar 18, 2019

… toolbar

Pull Request checklist

  • Quality: This PR builds and passes detekt/ktlint checks (A pre-push hook is recommended)
  • Tests: This PR includes thorough tests or an explanation of why it does not
  • Changelog: This PR includes a changelog entry or does not need one
  • Accessibility: The code in this PR follows accessibility best practices or does not include any user facing features

@ekager ekager requested a review from a team as a code owner March 18, 2019 22:33
pocmo
pocmo reviewed Mar 19, 2019
View reviewed changes
Copy link
Contributor

@pocmo pocmo left a comment

Choose a reason for hiding this comment

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

Is that something that the UX team has looked at? I haven't seen any loading state in the UX mocks yet. :(

@ekager ekager closed this Mar 19, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@pocmo pocmo pocmo left review comments

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@ekager @pocmo