search

KevinBatdorf / liquidslider

A Responsive jQuery Content Slider
161 stars 63 forks source link

includeTitle - false #111

Closed quma closed 9 years ago

quma commented 10 years ago

Hallo,

thx for the great slider. I think I found a bug, but I'm not sure.

includeTitle: false

Works not with:

dynamicTabs: false

Best regards

KevinBatdorf commented 10 years ago

Not sure if it's a bug because the only reason you would want to remove the title would be when you don't want duplicate content in the tabs. What is your reason for using these settings?

quma commented 10 years ago

Yes I get the Title double, but I use no tabs just cross links. I have a other question .. I don't like the extra generated divs like panel-container and panel-wrapper etc. Is it possible to specify witch html elements this script uses for the panel elements, like: use for: panel-container = "section.page" panel-wrapper = "section.subpage"

best regards

KevinBatdorf commented 10 years ago

No, it's only divs. I'm not sure using <section> tags is semantically correct here anyway. Adding the option would just add bloat.