Add loading screen after publishing, before success message #1647

New issue

Closed

opened 2018-06-20 00:02:31 +02:00 by tzarebczan · 7 comments

tzarebczan commented

2018-06-20 00:02:31 +02:00

(Migrated from github.com)

The Issue

After you click the publish button, the content get's unfocused (as if you were starting a new publish) and then a few seconds later, you get the modal with the success message. We should add a loading indicator during this process.

Expected Behaviour

Tell us what should happen.

Actual Behaviour

Tell us what happens instead.

System Configuration

LBRY Daemon version:
LBRY App version:
LBRY Installation ID:
Operating system:

Anything Else

Screenshots

## The Issue After you click the publish button, the content get's unfocused (as if you were starting a new publish) and then a few seconds later, you get the modal with the success message. We should add a loading indicator during this process. ### Expected Behaviour Tell us what should happen. ### Actual Behaviour Tell us what happens instead. ### Suggested Solutions Tell us your suggested solutions if you have any. ## System Configuration   - LBRY Daemon version: - LBRY App version: - LBRY Installation ID: - Operating system: ## Anything Else  ## Screenshots

neb-b commented

2018-07-02 19:06:04 +02:00

(Migrated from github.com)

Just going to disable the file input for now. In the future we will add some loading styling on the confirmation modal

tzarebczan commented

2018-09-17 18:17:15 +02:00

(Migrated from github.com)

A user just ran into what he thought was a bug because the publish took longer than expected (will happen with larger files). I think a loading screen would have at least let them know something was in progress.

neb-b commented

2018-10-16 19:54:54 +02:00

(Migrated from github.com)

I think for now we can just add the spinner component next to the button when it's publishing.

shahsank3t commented

2019-10-10 21:33:09 +02:00

(Migrated from github.com)

@tzarebczan @seanyesmunt - I would like to contribute to this issue if it's still open.

jessopb commented

2019-10-10 22:10:21 +02:00

(Migrated from github.com)

@shahsank3t Thanks! I'm about to PR my publish changes for llbrytv which somewhat deal with this issue.