signup form uses colors for input validation feedback

VERIFIED FIXED in 0.7

Status

P1
normal
VERIFIED FIXED
9 years ago
9 years ago

People

(Reporter: hello, Assigned: hello)

Tracking

unspecified
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Assignee)

Description

9 years ago
It was a quick hack, and needs to be replaced with something more accessible.  Not everyone can see red/green.
Blocks: 511688
Target Milestone: --- → 0.7

Updated

9 years ago
Priority: -- → P1
(Assignee)

Updated

9 years ago
Priority: P1 → P2

Updated

9 years ago
Priority: P2 → P1
Assignee: nobody → thunder
(Assignee)

Comment 1

9 years ago
Created attachment 402778 [details] [diff] [review]
Use text (in addition to colors)

Patch also includes some extra unrelated stuff to implement the start over bubble.
Attachment #402778 - Flags: review?(edilee)
(Assignee)

Comment 2

9 years ago
I should also note that the patch doesn't fix all of the problems we currently have with error reporting in about:weave, or all the instances where we use colors in fields.  But it's a definite improvement over what we currently do.
http://hg.mozilla.org/labs/weave/rev/2849b32ac6b5
Add initial error reporting and "available" tagging. Also add "start over" functionality and change some custom settings display.
Status: NEW → RESOLVED
Last Resolved: 9 years ago
Resolution: --- → FIXED
Attachment #402778 - Flags: review?(edilee) → review+
Comment on attachment 402778 [details] [diff] [review]
Use text (in addition to colors)

http://hg.mozilla.org/labs/weave/rev/40eb7be53737
Wire up "custom" inputs to set/read and show/hide server url box.
http://hg.mozilla.org/labs/weave/rev/fef981c73dd5
Reload the page instead of ending up with nothing when clicking "cancel" from start over.
http://hg.mozilla.org/labs/weave/rev/31f470639028
Add some account creation error handling, but it's still pretty clunky with username detection and alerts for other stuff.

Comment 7

9 years ago
Verified fix on weave 1.3b5.
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.