Skip to content

Conversation

jhermsen
Copy link

...880: Nested accordion icon padding isn't reset.

Added accordion-header-noicons css class to force padding on nested accordion.

@jhermsen
Copy link
Author

Ticket reference: http://bugs.jqueryui.com/ticket/7880

@scottgonzalez
Copy link
Member

Thanks, but this needs tests and must conform to our coding standards. The new class also doesn't get removed on destroy.

@scottgonzalez
Copy link
Member

This breaks in the opposite direction now. If you have an accordion with icons inside an accordion without icons, there's no padding on the accordion with icons.

@jhermsen
Copy link
Author

jhermsen commented Dec 2, 2011

Hi Scott,

Made a change to the style definitions for this problem.
Can you please tell me if there is instructions on making the tests you
mentioned in the first comment?
Then I can do more complete posts the next time. :)

Regards,
John

2011/11/29 Scott Gonzlez <
reply@reply.github.com

This breaks in the opposite direction now. If you have an accordion with
icons inside an accordion without icons, there's no padding on the
accordion with icons.


Reply to this email directly or view it on GitHub:
#525 (comment)

@scottgonzalez
Copy link
Member

Thanks, but this just breaks in the other direction again. This was fixed by moving the ui-accordion-icons class to the headers.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants