739155 - [UI] Invalid HTML for Bugzilla pop-ups

Status: VERIFIED FIXED

(Socorro :: Webapp, task)

Description

[Schalk Neethling [:espressive]]

On tcbs the code for the bug_ids_expanded_list, also reused on individual crash pages, incorrectly nests an unordered list inside a definition list leading to additional validation errors for this page.

Comment 1

[Schalk Neethling [:espressive]]

Sent pull request :: https://github.com/mozilla/socorro/pull/461

TEST CASE
---------

Head over to top crashers by version and right click on a 'Bugzilla IDs' content cell. In Firebug or FF dev tools, expand the <td> and look for a div with the class bug_ids_expanded_list. Expand this div and inside, the ul with class bug_ids_expanded, should no longer be wrapped with a <dl> 

This can also be tested on individual crash reports that have related bugzilla bug links. No functionality should be affected by this change.

Comment 2

[[github robot]]

Commit pushed to master at https://github.com/mozilla/socorro

https://github.com/mozilla/socorro/commit/624992e029ef46ad60ec1f2f31cad64be35b7a28
Merge pull request #461 from ossreleasefeed/invalid-html-bugzilla-popup-739155

removed wrapping definition list fixes bug 739155

Comment 3

[Matt Brandt [:mbrandt]]

Attachment: qa - verified

QA verified on stage -