Ticket #2529881 (assigned enhancement)

Reporter


bzbugs
Opened: 06/2/09
Last modified: 01/26/11
Status: assigned
Type: enhancement

Owner


Gopal Venkatesan
Target Release:
Priority: P4 (low)
Summary: [bz 2572879] Carousel LIs do not share a class, also one of the Carousel's CSS rule refers to LI instead of referencing a (generic) item
Description:

The only way to style a carousel's item/LI is by accessing the html node directly:

.myCarousel li {
width:644px;
}

But if each carousel item contains more LIs, then all LIs end up having a width of 644px.

Trying...

.myCarousel li.yui-carousel-item-selected{
width:644px;
}

...messes up the width calibration of the carousel.

One solution is to add a wrapper and then access the node...

.myCarousel .spotlight .anyWrapper li {
width:auto;
}

Still my concerns are:
1. More markup required by the user,
2. Is it safe to assume users will figure out the solution?
3. Styling a node directly just feels iffy in general.

Maybe add one class to all item/LIs?

Also notice I have to override styles being applied to all LIs inside a carousel to just to get default behavior each time I want to add a list to my carousel. Here's another default style I had to
reset:

.myCarousel .spotlight .anyWrapper li {
width:auto;
display:block;
}

So there's a user convenience as well as a default behavior factor.

Type: enhancement Observed in Version:
Component: None Severity: S3 (normal)
Assigned To: Gopal Venkatesan Target Release:
Location: Library Code Priority: P4 (low)
Tags: Relates To:
Browsers: N/A
URL:
Test Information:

Change History

George

YUI Developer

Posted: 06/2/09
  • status changed from new to accepted

Gopal Venkatesan

YUI Developer

Posted: 06/4/09
  • milestone changed from 2.NEXT to 2.8.0
  • priority changed from P3 (normal) to P4 (low)

George

YUI Developer

Posted: 08/5/09
  • milestone changed from 2.8.0 to 2.NEXT

deferring from 2.8 per bug review with Gopal

Gopal Venkatesan

YUI Developer

Posted: 01/7/11
  • milestone changed from 2.NEXT to FUTURE

George

YUI Developer

Posted: 01/26/11