Closed Bug 993920 Opened 10 years ago Closed 10 years ago

Collapse/Expand all comments button not working

Categories

(Bugzilla :: User Interface, defect)

4.5.2
defect
Not set
normal

Tracking

()

RESOLVED DUPLICATE of bug 974411

People

(Reporter: bePolite, Unassigned)

References

Details

(Keywords: regression)

When I click on Collapse/Expand all comments, I get some javascript errors

>TypeError: Cannot set property 'innerHTML' of null
on  Like 54 of js/comments.js:54
>>link.innerHTML = "[+]";
This is a regression due to bug 40896. The culprit is this line:

  <pre id="comment_preview_text" class="bz_comment_text"></pre>

toggle_all_comments() calls

  var comments = YAHOO.util.Dom.getElementsByClassName('bz_comment_text');

to get all comments to collapse, but the line above introduced a field with an id which doesn't contain an integer, making the JS code to fail as all comments were supposed to have an integer in their id, of the form:

  var link = document.getElementById('comment_link_' + id);
Depends on: 40896
Keywords: regression
OS: Linux → All
Hardware: x86_64 → All
Target Milestone: --- → Bugzilla 5.0
Version: unspecified → 4.5.2
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → DUPLICATE
Target Milestone: Bugzilla 5.0 → ---
You need to log in before you can comment on or make changes to this bug.