Pagination: false does not seem to work as described.

Splidejs / splide

Splide is a lightweight, flexible and accessible slider/carousel written in TypeScript. No dependencies, no Lighthouse errors.

https://splidejs.com

MIT License

4.83k stars 418 forks source link

Pagination: false does not seem to work as described. #1269

Open msamricth opened 7 months ago

msamricth commented 7 months ago

Checks

[X] Not a duplicate.
[X] Not a question, feature request, or anything other than a bug report directly related to Splide. Use Discussions for these topics: https://github.com/Splidejs/splide/discussions

Version

Description

using 'pagination: false' does not prevent the dot navigation from loading. I have tried both in the options in my JS and as a data attribute.

Reproduction Link

No response

Steps to Reproduce

Include pagination: false in your options. Add one element inside your slides that is position absolute.

Expected Behaviour

I am expecting there to be no html generated for the dots/pagination.

Webberjo commented 7 months ago

Are you adding your own <ul class="splide__pagination"></ul> placeholder in the HTML?

msamricth commented 7 months ago

No. Not sure why one would include that when they are specifically trying to avoid having the pagination.

On Mon, Jan 29, 2024, 9:19 AM Joseph Webber @.***> wrote:

Are you adding your own
placeholder in the HTML?

— Reply to this email directly, view it on GitHub https://github.com/Splidejs/splide/issues/1269#issuecomment-1914789735, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAJFCIUX5AAXWUEPEZFXFNDYQ6VYDAVCNFSM6AAAAABCJCAVIOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSMJUG44DSNZTGU . You are receiving this because you authored the thread.Message ID: @.***>

Webberjo commented 7 months ago

I'm unable to reproduce your issue using the steps you provided. https://jsfiddle.net/Webberjo/z9p435cu/1/