Opened 8 years ago

Closed 8 years ago

Last modified 8 years ago

#7880 closed bug (fixed)

Nested accordion icons

Reported by: j.hermsen Owned by:
Priority: minor Milestone: 1.9.0
Component: ui.accordion Version: 1.8.16
Keywords: Cc:
Blocked by: Blocking:

Description

A little styling issue if you have nested accordions.

if you have a parent accordion with icons and the sub accordion does not have icons the header will inherit the padding of the parent accordion so more blank space on the left.

I'd suggest adding a css class for example "ui-accordion-noicons" incase icons==false.

Change History (3)

comment:1 Changed 8 years ago by j.hermsen

Not sure if I have the correct link here, but did a fork/fix and pull request.

https://github.com/jquery/jquery-ui/pull/525

Before/After screenshot of the issue:

https://github.com/jhermsen/shared/blob/master/issues/accordionIssue.png

Last edited 8 years ago by j.hermsen (previous) (diff)

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

Resolution: fixed
Status: newclosed

Accordion: Moved the ui-accordion-icons class to the headers to properly style nested accordions with varying settings of icons and no icons. Fixes #7880 - Nested accordion.

Changeset: 5a55d68eca60b52cbd1ae1ab48a6c9b1f8817ef1

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

Summary: Nested accordionNested accordion icons
Note: See TracTickets for help on using tickets.