Add more columns per row for carousel

  • Resolved metdark

    (@metdark)


    6 years, 7 months ago

    Please help, I have issues with carousel’s columns display.

    Can columns per row for carousel be more than 5?
    I think it’s better if you don’t limit this number and let the user decide.

    For example I am using your carousel as a logo image slider.
    So I can actually fit more than 5 logos.

    Is it possible not to have dynamic id for the carousel?
    For example in lae-carousel-5b4d5c6097bcf, 5b4d5c6097bcf is not constant and keeps changing.

    I’ve been trying to get the 7 columns display.
    This is not working
    a) $( “.lae-carousel” ).data( “settings” ).display_columns = 7;
    Do you have a way to change this column display through jQuery on my WordPress theme?

    Only changing your codes in addons-for-elementor/includes/widgets/carousel.php works
    I change it to 10 for more options later.

            $this->add_control(
            'display_columns',
            [
                'label' => __('Columns per row', 'livemesh-el-addons'),
                'type' => Controls_Manager::NUMBER,
                'min' => 1,
                'max' => 10,
                'step' => 1,
                'default' => 3,
            ] /*'max' => 5,*/
        );

    Thanks!

Viewing 8 replies - 1 through 8 (of 8 total)
  • Plugin Author livemesh

    (@livemesh)

    I have changed the code on my dev build to enable support for 10 columns for the carousel. Will release the change in the upcoming update.

    Also, you can change the line number 521 of the same file in the latest update to – 

    $output = '<div id="lae-carousel-' . $this->get_id() . '" class="lae-carousel lae-container" data-settings=\'' . wp_json_encode($carousel_settings) . '\'>';

    to get an unique ID which stays across multiple invocations of the page. This change too will be part of upcoming update. Thanks for your suggestions.

    Thread Starter metdark

    (@metdark)

    Thanks!

    No problem but you may want to remove the column limit.
    Is there a reason for the 10 column limit?

    Plugin Author livemesh

    (@livemesh)

    Only reason the limit exists – I could not think of an usecase where more than 10 would be required ?? I still want to hold onto the limit but the question is what should be max for real scenarios where the carousels are going to be useful?

    Thread Starter metdark

    (@metdark)

    I was using it for logos which are as small as icons.

    Like for example past and present clients for the website service.
    It could be more than 10 clients.

    I understand the more server calls for the logos or the accumulated logos size, the slower it will be.

    To be honest after 2 weeks, I can’t remember whether this is the limit shown or the total limit of the row.

    Plugin Author livemesh

    (@livemesh)

    I changed the limit to 10 in the update released today. However, based on your description of yours and to keep it flexible, I have changed the column limit (items displayed per row – there is no limit on total number of items for a carousel) to 25. Will be part of next update.

    Thread Starter metdark

    (@metdark)

    Thanks!
    I also want to add that your plugin truly enhanced Elementor.

    Thread Starter metdark

    (@metdark)

    Yeap, I just left a review!

    Plugin Author livemesh

    (@livemesh)

    That was quick. Thank you ??

Viewing 8 replies - 1 through 8 (of 8 total)
  • The topic ‘Add more columns per row for carousel’ is closed to new replies.

Tags