Closed
Bug 452733
Opened 16 years ago
Closed 16 years ago
[Classification] should not be next to bug ID and short_desc in the show_bug header
Categories
(Bugzilla :: User Interface, defect)
Tracking
()
VERIFIED
FIXED
Bugzilla 3.2
People
(Reporter: mkanat, Assigned: mkanat)
References
Details
Attachments
(1 file, 2 obsolete files)
814 bytes,
patch
|
LpSolit
:
review+
|
Details | Diff | Splinter Review |
A more appropriate place for [Classification] would be next to Product (on the right).
Assignee | ||
Comment 1•16 years ago
|
||
Okay, here we go. I also added an id so some day we can add JS to the Product box to change the text appropriately.
Comment 2•16 years ago
|
||
Comment on attachment 336015 [details] [diff] [review] v1 https://landfill.bugzilla.org/mkanat4/show_bug.cgi?format=multiple&id=2323 shows the classification before the product. Need to pick one way and stay with it. Is the classification really needed here anyway? We use classifications on bmo, but we purposely don't show the classifications in the UI, as it's not necessary. Just wondering if it's necessary at all...
Attachment #336015 -
Flags: review?(reed) → review-
Assignee | ||
Comment 3•16 years ago
|
||
On the editing page, there's "Product:" and then an editing box. On the show-multiple page, there's no edit box. I think the difference thus makes sense.
Comment 4•16 years ago
|
||
I'm with reed. We don't need the [Classification] to be displayed at all in show_bug.cgi.
Assignee | ||
Comment 5•16 years ago
|
||
Okay, I can just remove it, I have no problem with that.
Assignee | ||
Comment 6•16 years ago
|
||
Okay, this just removes it.
Attachment #336015 -
Attachment is obsolete: true
Attachment #336126 -
Flags: review?(LpSolit)
Comment 7•16 years ago
|
||
Comment on attachment 336126 [details] [diff] [review] v2 r=LpSolit
Attachment #336126 -
Flags: review?(LpSolit) → review+
Updated•16 years ago
|
Flags: approval3.2+
Flags: approval+
Comment 8•16 years ago
|
||
Comment on attachment 336126 [details] [diff] [review] v2 Please remove it from the multiple buglist view, too.
Comment 9•16 years ago
|
||
Remove it from show-multiple.html.tmpl, too.
Attachment #336169 -
Flags: review?(mkanat)
Assignee | ||
Comment 10•16 years ago
|
||
Comment on attachment 336169 [details] [diff] [review] patch - v2.1 This patch belongs on a separate bug.
Attachment #336169 -
Flags: review?(mkanat) → review-
Assignee | ||
Updated•16 years ago
|
Attachment #336169 -
Attachment is obsolete: true
Assignee | ||
Comment 11•16 years ago
|
||
tip: 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.129; previous revision: 1.128 done 3.2: 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.125.2.4; previous revision: 1.125.2.3 done
Severity: normal → minor
Status: ASSIGNED → RESOLVED
Closed: 16 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•