Error with {TRIGGERINDEX}

  • Resolved abel1011

    (@abel1011)


    5 months, 1 week ago

    Hello, I hope you’re well. It seems there is an issue with the calculation of {TRIGGERINDEX} when using interaction layers. Instead of calculating the position based on the custom selector, it calculates it based on the element where the interaction layer is configured.

    When working with static elements, this is not a problem since both values can match. However, when using a repeater builder, where there are intermediate elements, the issue arises.

    I was reviewing the code, and I noticed that the gspb_trigger_inter_Actions function receives both the element with the interaction layer configuration (in my case, the repeater builder) and the one selected by the custom selector (the li inside the repeater). However, when passing this to gspb_execute_inter_Actions, instead of passing the element selected by the custom selector (the li), it passes the element where the interaction layer was configured (the main div of the repeater builder). This causes the {TRIGGERINDEX} to always be 0.

    I look forward to your support with this. Best regards.

Viewing 6 replies - 1 through 6 (of 6 total)
  • Plugin Author wpsoul

    (@wpsoul)

    please provide screenshot of your setup. Please note that triggerindex calculates it based on Trigger, not Action selector. So, if you need to add it to custom action selector, you need to add also custom trigger selector

    • This reply was modified 5 months ago by wpsoul.
    Thread Starter abel1011

    (@abel1011)

    Hello, this is the image.

    By the way, great job with the Element Block, it’s fantastic. I noticed that you have dynamic repeatable as a content option; is it possible that you also include the query loop? That would allow for cleaner layouts. Regards

    Plugin Author wpsoul

    (@wpsoul)

    I see what you mean. This will be fixed. You will not need to set custom action selector for such kind of interactions. it will be added to trigger

    Thread Starter abel1011

    (@abel1011)

    Thanks for the confirmation, with the Greenlight elements these configurations will be cleaner and simpler. I hope you can add the query loop as an option inside the elements soon. I would also like to request to include the Post Content and Post Formatted Content within the “Post Data” options in the Dynamic Text Value, as currently it only allows, title, slug, etc. The Meta getter adds 2 divs and I would like to have a cleaner style.

    Greetings and keep up the excellent work.

    Plugin Author wpsoul

    (@wpsoul)

    Interaction layers should be fixed in current update

    Plugin Author wpsoul

    (@wpsoul)

    I am closing for now, reopen if it will not work

Viewing 6 replies - 1 through 6 (of 6 total)
  • You must be logged in to reply to this topic.