Change setting to accept JSON

Marking ✨ Possibility to make manual slider settings. Closed: duplicate as duplicate since this has more details.

I’m working on a merge request.

That sound great Mathew we really appreciate this contribution! We do want to keep the config we've exposed as fields we will be adding a lot more options but having the ability to add your own config as JSON sounds really great to us.

I've submitted the MR but I understand that the changes I made don't align with your plan. My approach was to replace existing config fields with a single JSON field. Alternatively, perhaps we could add a field like "Config mode" that lets the the user can choose between defining config as JSON or as fields. Fields would be displayed conditionally based on that decision.

Alternatively, perhaps we could add a field like "Config mode" that lets the the user can choose between defining config as JSON or as fields. Fields would be displayed conditionally based on that decision.

That approach sounds good, if you add that to your merge request I'd be happy to review and merge. Thanks for your work it is appreciated!

grasmash → closed merge request !13

grasmash → reopened merge request !13

I created a new merge request which is building on the one in #8. There is an optional Config JSON field in the settings, if it has a valid JSON object, Tiny slider settings are overwritten by the JSON.

I extended my merge request. There is an Advanced mode toggle which enables the free text field and hides other fields. A slight improvement on the UI.

The MR failed and the patch is not applying to 1.1.x-dev.

This is the merge request: https://git.drupalcode.org/project/tiny_slider/-/merge_requests/15

Thanks Matthew for getting this started and Endre for bringing it over the line, great contributions!

Automatically closed - issue fixed for 2 weeks with no activity.

I'm attempting to use the advanced mode feature and manually override configuration with json settings. After saving the view, the json settings field is set to null and the view is unchanged. Although, the advanced mode checkbox remains checked. Am I using this correctly?

Here are the json settings I'm adding:

[
    default_row_class: true,
    center: false,
    mode: carousel,
    prevButton: '.prev',
    nextButton: '.next',
    nav: true,
    navPosition: 'bottom',
    loop: true,
    items: 1,
    gutter: 0,
    slideBy: 1,
    autoplay: true,
    autoplayTimeout: 12000,
    autoplayHoverPause: true,
    itemsDesktop: 1,
    itemsMobile: 1,

    ]

@peter your json is invalid. Try this:

{
"default_row_class": true,
"center": false,
"mode": "carousel",
"prevButton": ".prev",
"nextButton": ".next",
"nav": true,
"navPosition": "bottom",
"loop": true,
"items": 1,
"gutter": 0,
"slideBy": 1,
"autoplay": true,
"autoplayTimeout": 12000,
"autoplayHoverPause": true,
"itemsDesktop": 1,
"itemsMobile": 1
}

see https://jsonlint.com/