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

add ProgressBar with new Toolbar to show loading #64

Closed
wants to merge 2 commits into from

Conversation

inthewaves
Copy link
Member

@inthewaves inthewaves commented Aug 23, 2020

On hold until #72 is looked at

A ProgressBar is displayed when a page is rendering; the ProgressBar is
hidden after the page has been displayed to the screen. In order to make
the ProgressBar show under the ActionBar, we use our own Toolbar
integrated into the Activity's layout instead of the one provided by the
system. (This also gives us a bit more control over styling.)

The ProgressBar is set to indeterminate. Zooming in does not show the
ProgressBar.

Closes #29

@inthewaves inthewaves changed the title add loading bar and use custom Toolbar add ProgressBar with app's Toolbar to show loading Aug 23, 2020
@inthewaves inthewaves changed the title add ProgressBar with app's Toolbar to show loading add ProgressBar with new Toolbar to show loading Aug 23, 2020
@inthewaves inthewaves force-pushed the progress-bar-pr branch 3 times, most recently from 9c3f204 to 7df800d Compare August 25, 2020 04:42
A ProgressBar is displayed when a page is rendering; the ProgressBar is
hidden after the page has been displayed to the screen. In order to make
the ProgressBar show under the ActionBar, we use our own Toolbar
integrated into the Activity's layout instead of the one provided by the
system. (This also gives us a bit more control over styling.)

The ProgressBar is set to indeterminate. Zooming in does not show the
ProgressBar.
@inthewaves
Copy link
Member Author

Redid this at https://github.com/inthewaves/PdfViewer/tree/fragments-revamp-theme based on the Fragments branch, so I'll wait until Fragments are ready before opening a new PR

@inthewaves inthewaves closed this Aug 30, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

indicate when a page is loading
1 participant