Closed Bug 363556 Opened 18 years ago Closed 18 years ago

No obvious way to log in to view/change a bug when you're not logged in.

Categories

(Bugzilla :: User Interface, defect)

2.23.3
defect
Not set
major

Tracking

()

RESOLVED FIXED
Bugzilla 3.0

People

(Reporter: justdave, Assigned: LpSolit)

References

Details

Attachments

(1 file, 2 obsolete files)

Bug 95923 introduced showing the bug as text-only (no edit form) when you're not logged in.  There's two things we need to fix in the UI to make this actually usable:

a) There need to be a conspicuous notice somewhere on the page (the space where the comment form normally goes would be a good spot for it) saying "You can log in to add a comment or edit this bug" ("log in" could probably be linked there).

b) The existing login links on the page all go to the index page instead of back to the bug after you log in.  This can probably be dealt with by having the log in link in a) link to the bug after logging in.
Depends on: 95923
Attached patch patch, v1 (obsolete) — Splinter Review
Implement justdave's suggestion.
Assignee: ui → LpSolit
Status: NEW → ASSIGNED
Attachment #248399 - Flags: review?(justdave)
Flags: blocking3.0?
Attached patch patch, v1 (obsolete) — Splinter Review
Let's try with the correct file :)
Attachment #248399 - Attachment is obsolete: true
Attachment #248401 - Flags: review?(justdave)
Attachment #248399 - Flags: review?(justdave)
Comment on attachment 248401 [details] [diff] [review]
patch, v1

>+            <legend>Warning</legend>

Let's do "Note" instead of "Warning".  "Warning implies that there's a problem, and if they're not intending to edit the bug anyway, there's no problem. ;)

>+              You are not authenticated! You first have to
>+              <a href="show_bug.cgi?id=[% bug.bug_id %]&amp;GoAheadAndLogIn=1">log in</a><br>
>+              to comment or do changes in this bug.

Again, the "You are not authenticated!" goes with the "Warning" context, and we're not wanting to get people all riled up. ;)  Let's make this "You need to <a>log in</a> before you can comment on or make changes to this bug."
Attachment #248401 - Flags: review?(justdave) → review-
Comment on attachment 248401 [details] [diff] [review]
patch, v1

>+          <fieldset>

Does the fieldset need an id so it can be styled by a skin?
Attached patch patch, v1.1Splinter Review
Attachment #248401 - Attachment is obsolete: true
Attachment #248406 - Flags: review?(justdave)
Attachment #248406 - Flags: review?(justdave) → review+
Flags: approval?
Flags: blocking3.0? → blocking3.0+
Flags: approval? → approval+
Checking in template/en/default/bug/edit.html.tmpl;
/cvsroot/mozilla/webtools/bugzilla/template/en/default/bug/edit.html.tmpl,v  <--  edit.html.tmpl
new revision: 1.93; previous revision: 1.92
done
Status: ASSIGNED → RESOLVED
Closed: 18 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: