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)
Tracking
()
RESOLVED
FIXED
Bugzilla 3.0
People
(Reporter: justdave, Assigned: LpSolit)
References
Details
Attachments
(1 file, 2 obsolete files)
(deleted),
patch
|
justdave
:
review+
|
Details | Diff | Splinter Review |
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.
Assignee | ||
Comment 1•18 years ago
|
||
Implement justdave's suggestion.
Assignee | ||
Updated•18 years ago
|
Flags: blocking3.0?
Assignee | ||
Comment 2•18 years ago
|
||
Let's try with the correct file :)
Attachment #248399 -
Attachment is obsolete: true
Attachment #248401 -
Flags: review?(justdave)
Attachment #248399 -
Flags: review?(justdave)
Reporter | ||
Comment 3•18 years ago
|
||
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 %]&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-
Reporter | ||
Comment 4•18 years ago
|
||
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•18 years ago
|
||
Attachment #248401 -
Attachment is obsolete: true
Attachment #248406 -
Flags: review?(justdave)
Reporter | ||
Updated•18 years ago
|
Attachment #248406 -
Flags: review?(justdave) → review+
Assignee | ||
Updated•18 years ago
|
Flags: approval?
Reporter | ||
Updated•18 years ago
|
Flags: blocking3.0? → blocking3.0+
Reporter | ||
Updated•18 years ago
|
Flags: approval? → approval+
Assignee | ||
Comment 6•18 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
Closed: 18 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•