Solved Fullwidth Slider showing numbers instead of title

Hey Daniel, you have the following code in your child theme's makan.js file:

Code:

  // Hero Slider Pagination
  function heroSlider() {
    $('.et-pb-controllers').addClass('loaded');

    $('.et-pb-controllers a').each(function() {
      // Slide 1
      if ($(this).text() == '1') {
        var title = $('.et_pb_slide_0 .slide-title').text();

        $(this).text(title);
      }
      // Slide 2
      if ($(this).text() == '2') {
        var title = $('.et_pb_slide_1 .slide-title').text();

        $(this).text(title);
      }
      // Slide 3
      if ($(this).text() == '3') {
        var title = $('.et_pb_slide_2 .slide-title').text();

        $(this).text(title);
      }
      // Slide 4
      if ($(this).text() == '4') {
        var title = $('.et_pb_slide_3 .slide-title').text();

        $(this).text(title);
      }
      // Slide 5
      if ($(this).text() == '5') {
        var title = $('.et_pb_slide_4 .slide-title').text();

        $(this).text(title);
      }
      // Slide 6
      if ($(this).text() == '6') {
        var title = $('.et_pb_slide_5 .slide-title').text();

        $(this).text(title);
      }
    });
  }

It's converting the control dots to titles in your slider, but is only coded to go up to slide 6. after that the default text of the control dot (e.g. "7", "8", etc, is used).

While you could solve this by adding additional lines for each slide, you might find it more convenient to replace this function with a loop that can handle any number of slides - something like this:

Code:

function heroSlider() {
  $('.et-pb-controllers').addClass('loaded');

  $('.et-pb-controllers a').each(function(index) {
    var slideIndex = index + 1; // Add 1 because .et_pb_slide indexes start from 0

    var title = $('.et_pb_slide_' + index + ' .slide-title').text();
    $(this).text(title);
  });
}

 

You're welcome, Daniel