Closed
Bug 592382
Opened 14 years ago
Closed 14 years ago
Do account + server validation in wizard on a delay
Categories
(Firefox :: Sync, defect)
Tracking
()
VERIFIED
FIXED
People
(Reporter: mconnor, Assigned: philikon)
References
Details
Attachments
(1 file)
6.29 KB,
patch
|
mconnor
:
review+
|
Details | Diff | Splinter Review |
No description provided.
Assignee | ||
Comment 1•14 years ago
|
||
We should also do the custom server validation on a delay to avoid blocking the UI.
Assignee | ||
Comment 2•14 years ago
|
||
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.
Assignee | ||
Comment 3•14 years ago
|
||
(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
Assignee | ||
Comment 4•14 years ago
|
||
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)
Reporter | ||
Comment 5•14 years ago
|
||
Comment on attachment 479066 [details] [diff] [review] v1 I miss timed textboxes.
Attachment #479066 -
Flags: review?(mconnor) → review+
Assignee | ||
Comment 6•14 years ago
|
||
http://hg.mozilla.org/services/fx-sync/rev/23caf720a4a1
Status: NEW → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
Updated•14 years ago
|
Status: RESOLVED → VERIFIED
Updated•6 years ago
|
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.
Description
•