Accordion: classNames should be default

[neojp]

Theme related classNames should be added to the accordions by default.

$(container).addClass("ui-accordion-container ui-accordion");

// add classNames and spans to header
options.headers.addClass('ui-accordion-link').prepend('<span class="ui-accordion-left"></span>').append('<span class="ui-accordion-right"></span>');

[neojp]

UI Accordion: add classnames, and spans for themes

[neojp]

Talking about a theming issue, headers should be relative positioned.

ul.ui-accordion-container li a.ui-accordion-link { position: relative }

[neojp]

accordion flora theme, position relative added to headers

[scott.gonzal]

Why are both .ui-accordion and .ui-accordion-container needed if they're applied to the same element?

Also, why is relative positioning required? I'm asking because this was actually removed in [4420] to fix #2127.

[neojp]

postion:relative; is not that necessary if you don't append the header spans. If they are not positioned inside a reltive they will appear on the top right corner of the document.

Regarding this,

$(container).addClass("ui-accordion-container ui-accordion");

I'm not sure why the accordion plugin uses 2 different classNames for the same element, since it never uses one. I think we could get rid of one and change the css styles to match the className choosen.

[joern]

The second class got removed, other issues are now on the UI bugtracker.