Opened 9 years ago

Closed 9 years ago

Last modified 9 years ago

#6720 closed bug (fixed)

Accordion: ol numbering bug

Reported by: chyi1235 Owned by:
Priority: minor Milestone: 1.8.8
Component: ui.accordion Version: 1.8.6
Keywords: Cc:
Blocked by: Blocking:

Description

<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.

Change History (9)

comment:1 Changed 9 years ago by Scott González

Resolution: worksforme
Status: newclosed

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

comment:2 Changed 9 years ago by chyi1235

as mentioned, not under IE 8

comment:3 Changed 9 years ago by chyi1235

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

comment:4 Changed 9 years ago by Scott González

Resolution: worksforme
Status: closedreopened

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

comment:5 Changed 9 years ago by Scott González

Status: reopenedopen

comment:6 Changed 9 years ago by 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

comment:7 Changed 9 years ago by Scott González

Summary: ol numbering bugAccordion: ol numbering bug

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

Last edited 9 years ago by Scott González (previous) (diff)

comment:8 Changed 9 years ago by Scott González

Resolution: fixed
Status: openclosed

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

Thanks Tony Ross.

Changeset: dd89ea5ff30a87d756453a73651cd30ae658492a

comment:9 Changed 9 years ago by Scott González

Milestone: 1.91.8.8
Note: See TracTickets for help on using tickets.