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.
Created attachment 248399 [details] [diff] [review] patch, v1 Implement justdave's suggestion.
Assignee: ui → LpSolit
Status: NEW → ASSIGNED
Attachment #248399 - Flags: review?(justdave)
Created attachment 248401 [details] [diff] [review] patch, v1 Let's try with the correct file :)
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-
Comment on attachment 248401 [details] [diff] [review] patch, v1 >+ <fieldset> Does the fieldset need an id so it can be styled by a skin?
Created attachment 248406 [details] [diff] [review] patch, v1.1
Attachment #248406 - Flags: review?(justdave) → review+
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
Last Resolved: 12 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.