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.

Sign up for GitHub

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

Jump to bottom

Add loading dialog #125

Merged
merged 3 commits into from
Aug 17, 2022
Merged

Add loading dialog #125

merged 3 commits into from
Aug 17, 2022

Conversation

edgarmiro
Copy link
Contributor

@edgarmiro edgarmiro commented Aug 8, 2022 •
edited
Loading

Add loading dialog

This PR adds a loading dialog to avoid the black screen while copying the files.

Description

  • I moved the specific file methods to its own file in order to keep the code clean.
  • The (weight) task of copying files now is performed in another thread.
  • A progress dialog is shown until the copy files task is completed.

Sample

device-2022-08-08-163454.mp4

@edgarmiro
Copy link
Contributor Author

@alexbatalov Could you check this PR, please? 🙂

@alexbatalov alexbatalov merged commit 4eb5e39 into alexbatalov:main Aug 17, 2022
@edgarmiro edgarmiro mentioned this pull request Aug 24, 2022
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@edgarmiro @alexbatalov