Page MenuHomePhabricator

Step 1: Loading bar for saving (impact: low)
Closed, ResolvedPublic8 Estimated Story Points

Assigned To
Authored By
Charlie_WMDE
Nov 5 2019, 4:24 PM
Referenced Files
F31478859: Save_processing_mobile.png
Dec 17 2019, 10:59 AM
F31478813: Save_processing.png
Dec 17 2019, 10:59 AM
F31458545: fullModal.png
Dec 5 2019, 3:21 PM
F31455289: fullModal.png
Dec 3 2019, 2:32 PM
F31100617: saving.png
Nov 19 2019, 10:14 AM
F30389498: loading screen.png
Nov 5 2019, 4:24 PM

Description

When a user pushes the "publish" button to save their edit, the transmission of the data might not be immediate and thus the bridge would remain open until the data would be sent.

In order to avoid confusion for the user, who would expect an immediate closing of the modal and a visible change of information in the infobox, we should have a loading indicator so they know that their request is being processed until the modal is either closed on success or an error message appears (to be handled in a different ticket).

MOCKS

specs can be found here

Save_processing.png (486×500 px, 33 KB)

Save_processing_mobile.png (686×375 px, 37 KB)

Acceptance criteria

  • uses Initializing component (feel free to rename ;-))
    • this implies using the same timings as for the loading bar on init
  • add a semitransparent overlay (= termbox' Overlay with IndeterminateProgressBar) immediately after the user hits save
  • after clicking publish
    • the publish button becomes disabled (primaryprogressivedisabled)
    • the cancel button becomes disabled
  • remove global ESC handling (which may be readded in a dedicated story incl confirmation prompt not to lose data, ...)

Event Timeline

Restricted Application added a subscriber: Aklapper. · View Herald Transcript

notes: we'd need a transparent overlay that turns opaque after 1sec

Charlie_WMDE updated the task description. (Show Details)
darthmon_wmde renamed this task from Loading bar for saving to Step 1: Loading bar for saving.Dec 5 2019, 1:09 PM

TODO for Sarai and me: decide on whether the cancel button should also be disabled and what the timeout time for the saving time should be

Lydia_Pintscher renamed this task from Step 1: Loading bar for saving to Step 1: Loading bar for saving (impact: low).Dec 8 2019, 1:20 PM

Change 575292 had a related patch set uploaded (by Pablo Grass (WMDE); owner: Pablo Grass (WMDE)):
[mediawiki/extensions/Wikibase@master] bridge: allow interaction with Loading in storybook

https://gerrit.wikimedia.org/r/575292

Change 575546 had a related patch set uploaded (by Lucas Werkmeister (WMDE); owner: Pablo Grass (WMDE)):
[mediawiki/extensions/Wikibase@master] bridge: allow interaction with Loading in storybook

https://gerrit.wikimedia.org/r/575546

Change 575292 merged by jenkins-bot:
[mediawiki/extensions/Wikibase@master] bridge: allow interaction with Loading in storybook

https://gerrit.wikimedia.org/r/575292

Change 576314 had a related patch set uploaded (by Pablo Grass (WMDE); owner: Pablo Grass (WMDE)):
[mediawiki/extensions/Wikibase@master] bridge: improve Loading documentation

https://gerrit.wikimedia.org/r/576314

Change 576314 merged by jenkins-bot:
[mediawiki/extensions/Wikibase@master] bridge: improve Loading documentation

https://gerrit.wikimedia.org/r/576314

Charlie_WMDE claimed this task.
Charlie_WMDE moved this task from Verification to Done on the Wikidata-Bridge-Sprint-15 board.

Not in the AC but ideally the the overlay would not be affected by the scrolling. moving the ticket to done because this was not an AC but i will make an extra ticket for this for step n.

Change 575546 abandoned by Lucas Werkmeister (WMDE):
bridge: allow interaction with Loading in storybook

https://gerrit.wikimedia.org/r/575546