Skip to content

Conversation

@JorchCortez
Copy link
Contributor

New Feature: Multiple Improvements

Description

Added multiple improvements to the range slider, which allows us to match the value in the text box to the values in the slider in both directions. This means that if we modify the box, it updates the range, and vice versa.
Added an update to set default range values in range sliders if they exist to match the max and min values from the current list of elements.
Updated the cookie consent form to make sure it matches the requirements for Webflow when using Facebook pixels through the apps section on the page settings.

Script Details

  • Script Name: CMS Filter, Range Slider, CookieConsent
  • Purpose of Update: Multiple purposes, this is a large pr as it touches several issues.

This PR closes NONE current issues

Checklist

  • Feature has been tested locally with all relevant use cases.
  • Documentation has been updated (if necessary).
  • PR does not match another non-stale PR currently opened
  • PR name matches the format [ feature ]: Feature Name (versions separated by comma). More details here
  • PR's base is the develop branch.
  • Your Feature matches the standards laid out here

@JorchCortez JorchCortez requested a review from Copilot August 19, 2025 15:43

This comment was marked as outdated.

@JorchCortez JorchCortez requested a review from Copilot August 19, 2025 15:45

This comment was marked as outdated.

@JorchCortez JorchCortez requested a review from Copilot August 19, 2025 15:46
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot wasn't able to review any files in this pull request.


Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@JorchCortez JorchCortez merged commit b3f4527 into Develop Aug 19, 2025
@JorchCortez JorchCortez deleted the Updates-to-CMS-Filter branch August 19, 2025 15:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants