357534 - Add fieldsets in show_bug.cgi

Status: RESOLVED FIXED

(Bugzilla :: User Interface, enhancement)

Description

[Frédéric Buclin]

From our discussion on IRC:

(22:54:27) mkanat: LpSolit: Here's what I think:
(22:54:31) LpSolit: hmm, I'm fine in grouping 1 2 and 4
(22:54:37) mkanat: You combine 1, 2, and 4 into a single table.
(22:54:42) mkanat: Summary goes at the top, in a colspan.
(22:54:52) mkanat: And the single fieldset is called "Details."
(22:55:10) LpSolit: sounds good

1, 2 and 4 are references to http://landfill.bugzilla.org/qa30pg/attachment.cgi?id=350

Comment 1

[Frédéric Buclin]

Attachment: patch, v1

good luck...

Comment 2

[Frédéric Buclin]

Attachment: patch, v1.1

unbitrotten patch

Comment 3

[Max Kanat-Alexander]

Comment on attachment 243057 [details] [diff] [review]
patch, v1.1

Okay, this looks great in the UI, and it validates. It's such a big patch, I didn't actually do a code review of it (other than just looking it over briefly), but it actually *works* well, and that's what matters to me about the UI. :-)

Comment 4

[Frédéric Buclin]

Note to self: write "Depends on" instead of "Depends on" on checkin so that this label remains on a single line.

Comment 5

[Frédéric Buclin]

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.89; previous revision: 1.88
done
Checking in template/en/default/flag/list.html.tmpl;
/cvsroot/mozilla/webtools/bugzilla/template/en/default/flag/list.html.tmpl,v  <--  list.html.tmpl
new revision: 1.24; previous revision: 1.23
done