Closed Bug 240845 Opened 16 years ago Closed 6 years ago

Clarify on the login page that login is an email address.

Categories

(Bugzilla :: User Interface, enhancement)

enhancement
Not set

Tracking

()

RESOLVED FIXED
Bugzilla 5.0

People

(Reporter: irrevenant, Assigned: sjoshi)

References

Details

(Whiteboard: [applied to b.m.o])

Attachments

(1 file, 3 obsolete files)

User-Agent:       Mozilla/5.0 (Windows; U; Win98; en-US; rv:1.6) Gecko/20040206 Firefox/0.8
Build Identifier: Mozilla/5.0 (Windows; U; Win98; en-US; rv:1.6) Gecko/20040206 Firefox/0.8

At login, the page does not make it clear that the login required is an email
address.  Bugzilla is also the sort of site that people will tend to use
infrequently.  Since people have various logins for various sites, it would be a
handy to jog the memory as to what is required.

You could probably even just call the field something like "Registered email
address".

Reproducible: Always
Steps to Reproduce:
But it isn't necessarily an email address.  On most bugzilla sites it is, but it
doesn't have to be.  That statement was intentionally removed because some sites
don't use an email address as the login ID...
Not sure if we want this (see comment 1) but it seems to be applied to b.m.o
already, so attaching the patch for posterity.
Status: UNCONFIRMED → NEW
Ever confirmed: true
OS: Windows 98 → All
Hardware: PC → All
Whiteboard: [applied to b.m.o]
Target Milestone: --- → Future
Summary: Please clarify on the login page that login is an email address. → Clarify on the login page that login is an email address.
QA Contact: mattyt-bugzilla → default-qa
Target Milestone: Future → ---
Assignee: myk → ui
Comment on attachment 168488 [details] [diff] [review]
Patch

Just adding a reviewer.
Attachment #168488 - Flags: review?(dkl)
Comment on attachment 168488 [details] [diff] [review]
Patch

No, as said in comment 1, login != email address.
Attachment #168488 - Flags: review?(dkl) → review-
Well in that case , this stands Invalid for me.
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → INVALID
(In reply to Sunil Joshi from comment #6)
> Well in that case , this stands Invalid for me.

Depends on the value of the emailsuffix parameter. If it's empty, then login == email address.
Status: RESOLVED → REOPENED
Resolution: INVALID → ---
Attached patch Patch-v2 (obsolete) — Splinter Review
Oops !! Yes agree.
Assignee: ui → joshi_sunil
Attachment #168488 - Attachment is obsolete: true
Status: REOPENED → ASSIGNED
Attachment #782186 - Flags: review?(sgreen)
Comment on attachment 782186 [details] [diff] [review]
Patch-v2

That's exactly the opposite. If emailsuffix is in use, then you must *not* enter your email address. Also, please fix the indentation:

<tr>
  <th>
    <label>
    </label>
  </th>
  ...
Attachment #782186 - Flags: review?(sgreen) → review-
Attached patch Patch-v3 (obsolete) — Splinter Review
Attachment #782186 - Attachment is obsolete: true
Attachment #782189 - Flags: review?(sgreen)
Comment on attachment 782189 [details] [diff] [review]
Patch-v3

Review of attachment 782189 [details] [diff] [review]:
-----------------------------------------------------------------

Indentation is still incorrect. It should be two spaces for each level (including TT lines).
Attachment #782189 - Flags: review?(sgreen) → review-
Attached patch Patch-v4Splinter Review
With review comments.
Attachment #782189 - Attachment is obsolete: true
Attachment #783230 - Flags: review?(sgreen)
Attachment #783230 - Flags: review?(sgreen) → review+
Flags: approval?
Target Milestone: --- → Bugzilla 5.0
Flags: approval? → approval+
Committing to: bzr+ssh://bzr.mozilla.org/bugzilla/trunk/
modified template/en/default/account/auth/login.html.tmpl
Committed revision 8686.

Unfortunately, the commit log will forever refer to bug 899679. Very sorry about that.
See Also: → 899679
Status: ASSIGNED → RESOLVED
Closed: 6 years ago6 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.