Closed Bug 592382 Opened 14 years ago Closed 14 years ago

Do account + server validation in wizard on a delay

Categories

(Firefox :: Sync, defect)

x86
macOS
defect
Not set
normal

Tracking

()

VERIFIED FIXED

People

(Reporter: mconnor, Assigned: philikon)

References

Details

Attachments

(1 file)

      No description provided.
Blocks: 592375
We should also do the custom server validation on a delay to avoid blocking the UI.
This has gone away with the wizard revamp, specifically with the merging of the username and email field in bug 589979.

There's still one wart: when entering either the email address or server, the UI isn't updated until the corresponding textboxes lose focus.
(In reply to comment #2)
> There's still one wart: when entering either the email address or server, the
> UI isn't updated until the corresponding textboxes lose focus.

This behaviour also causes bug 599451. I'm morphing this bug into fixing these UI warts while making sure that we don't validate the server or account name on every character input.
Blocks: 599451
Summary: do username validation in wizard on a delay (don't f the server team) → Do account + server validation in wizard on a delay
Attached patch v1Splinter Review
Don't rely on onchange for the email and server fields, these are only called when the text fields lose focus. Instead perform the checks when the user stops typing for 1 second. This minimizes the frequency of server calls while not requiring the text fields to lose focus. If the timer is still active when the user hits advance, perform the check immediately.
Assignee: nobody → philipp
Attachment #479066 - Flags: review?(mconnor)
Comment on attachment 479066 [details] [diff] [review]
v1

I miss timed textboxes.
Attachment #479066 - Flags: review?(mconnor) → review+
http://hg.mozilla.org/services/fx-sync/rev/23caf720a4a1
Status: NEW → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
Status: RESOLVED → VERIFIED
Component: Firefox Sync: UI → Sync
Product: Cloud Services → Firefox
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: