Accordion: ol numbering bug

[chyi1235]

<ol> in the accordion content (under IE8 only) has been replaced. example can be seen here at ​http://www.ziweiuniversity.com/faq_bug.php

Trying clicking on first then second, back to first and second, and so on. Notice the li numbering system being replaced.

Looked like <li value=""> being updated by accordion.

[Scott González]

The numbers look fine to me. Tested in Safari, Firefox and Opera.

[chyi1235]

as mentioned, not under IE 8

[chyi1235]

sorry i mean as mentioned in my report, under IE 8

[Scott González]

sorry, missed that part when the first time I read the ticket.

[hypnotoad]

I have also encountered this issue within an FAQ section within a jQuery UI Accordion which has a similar structure to the following...

<ol>
  <li>
    <span class='faq_question'>Question...</span>
    <fieldset class='inactive' style='display:none'>
      <label>Answer<label>
      <span>The Answer...</span>
    </fieldset>
  </li>
  ....
  ....
</ol>

I also have the following jQuery script powering it...

$(".faq_question").click(function(){
  $(".faq_question").removeClass('selected').next().hide('slow');
  $(this).addClass('selected').next().show('slow');
});

My experience has been as follows. 1) Developed the web app, which was being imported into a corporate themed webpage via an iFrame for showing management the intended end result. No problem. 2) Moved the web application code into the body of the corporate themed webpage (removed the iFrame). The problem appeared.

The issue is only present in IE8, and does not appear when I tested with compatibility mode turned on. Furthermore, the issue always reverts the numbering of the <ol> to zero. However, I can manually click on the number and it will then display the correct number. I hope this helps.

Thanks

[Scott González]

I've narrowed this down to ​an .addClass() call in _clickHandler(). Removing this line causes the numbering to work correctly.

[Scott González]

Accordion: Start the accordion animation before changing classes. Fixes #6720 - Accordion: ol numbering bug.

Thanks Tony Ross.

Changeset: dd89ea5ff30a87d756453a73651cd30ae658492a

[Scott González]

Fixed in 1-8-stable in dd7aae2cc8789316592aaa5f191fa9bb546befa5.