Closed Bug 975204 Opened 7 years ago Closed 7 years ago

Comment Preview tabs broken in New bug page

Categories

(Bugzilla :: User Interface, defect)

4.5.2
defect
Not set
major

Tracking

()

RESOLVED FIXED
Bugzilla 5.0

People

(Reporter: mayhemer, Assigned: glob)

References

Details

(Keywords: regression)

Attachments

(2 files)

Attached image broken-preview.png
See the screenshot.
Summary: Comment Preview tabs broken with Dusk theme → Comment Preview tabs broken with Dusk theme in New bug page
Same with the Classic skin. This is a regression due to bug 40896, which affects all upstream skins.
Assignee: nobody → user-accounts
Severity: normal → major
Component: Sandstone/Mozilla Skin → User Accounts
Depends on: 40896
Keywords: regression
OS: Windows 7 → All
Product: bugzilla.mozilla.org → Bugzilla
QA Contact: default-qa
Hardware: x86_64 → All
Summary: Comment Preview tabs broken with Dusk theme in New bug page → Comment Preview tabs broken in New bug page
Target Milestone: --- → Bugzilla 5.0
Version: Production → 4.5.2
Assignee: user-accounts → glob
Component: User Accounts → User Interface
Status: NEW → ASSIGNED
Attached patch 975204_1.patchSplinter Review
Attachment #8381160 - Flags: review?(gerv)
Comment on attachment 8381160 [details] [diff] [review]
975204_1.patch

Assuming we are happy supporting only IE 8 or above with display: table-cell, then r=gerv.

Gerv
Attachment #8381160 - Flags: review?(gerv) → review+
Flags: approval?
Flags: approval? → approval+
Committing to: bzr+ssh://bjones%40mozilla.com@bzr.mozilla.org/bugzilla/trunk/
modified skins/standard/global.css
modified template/en/default/bug/comment.html.tmpl
Committed revision 8927.

note: i also tweaked the whitespace below the preview to make it more consistent when switching between tabs.
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.