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

RESOLVED FIXED in Bugzilla 3.0

Status

()

--
major
RESOLVED FIXED
12 years ago
12 years ago

People

(Reporter: justdave, Assigned: LpSolit)

Tracking

2.23.3
Bugzilla 3.0
Bug Flags:
approval +
blocking3.0 +

Details

Attachments

(1 attachment, 2 obsolete attachments)

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
(Assignee)

Comment 1

12 years ago
Created attachment 248399 [details] [diff] [review]
patch, v1

Implement justdave's suggestion.
Assignee: ui → LpSolit
Status: NEW → ASSIGNED
Attachment #248399 - Flags: review?(justdave)
(Assignee)

Updated

12 years ago
Flags: blocking3.0?
(Assignee)

Comment 2

12 years ago
Created attachment 248401 [details] [diff] [review]
patch, v1

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?
(Assignee)

Comment 5

12 years ago
Created attachment 248406 [details] [diff] [review]
patch, v1.1
Attachment #248401 - Attachment is obsolete: true
Attachment #248406 - Flags: review?(justdave)
Attachment #248406 - Flags: review?(justdave) → review+
(Assignee)

Updated

12 years ago
Flags: approval?
Flags: blocking3.0? → blocking3.0+
Flags: approval? → approval+
(Assignee)

Comment 6

12 years ago
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
Last Resolved: 12 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.