pr feedback: version

Author: matthewcsimpson

README.md

@@ -82,15 +82,17 @@ Multiple Scripts: Add as many scripts as you need to your project by referencing
 **Range sliders:** each script is **self-contained** (one tag). Use **`RangeSlider.js`** for custom thumbs or **`RangeSliderSimple.js`** for native thumbs only. Use jsDelivr (not raw `githubusercontent.com`, which often serves `text/plain` and blocks execution).
 
 ```
-<script src="https://cdn.jsdelivr.net/gh/TheCodeRaccoons/WebTricks@main/Dist/Functional/RangeSlider.js"></script>
+<script src="https://cdn.jsdelivr.net/gh/TheCodeRaccoons/WebTricks@1/Dist/Functional/RangeSlider.js"></script>
 ```
 
-Native-thumb variant:
+Native-thumb variant (`RangeSliderSimple` is **1.0.0** in source; pin further with a [release tag](https://github.com/TheCodeRaccoons/WebTricks/releases) or commit hash if you need an exact file):
 
 ```
-<script src="https://cdn.jsdelivr.net/gh/TheCodeRaccoons/WebTricks@main/Dist/Functional/RangeSliderSimple.js"></script>
+<script src="https://cdn.jsdelivr.net/gh/TheCodeRaccoons/WebTricks@1/Dist/Functional/RangeSliderSimple.js"></script>
 ```
 
+Prefer **`@1`**, a **semver release tag** (e.g. `@v1.0.0`), or a **commit SHA** over **`@main`** for production embeds so the script URL does not move unexpectedly.
+
 If a page uses **both** slider types, you may include **both** scripts; they use separate attribute namespaces (`wt-rangeslider-*` vs `wt-rangeslidersimple-*`).
 
 Ready to Use: Once imported, the scripts initialize automatically, provided the correct HTML attributes are in place.

__tests__/RangeSliderSimple.test.js

@@ -44,8 +44,19 @@ describe('RangeSliderSimple', () => {
         const right = wrapper.querySelector(
             '[wt-rangeslidersimple-element="input-right"]',
         );
+        const displayFrom = wrapper.querySelector(
+            '[wt-rangeslidersimple-display="from"]',
+        );
+        const displayTo = wrapper.querySelector(
+            '[wt-rangeslidersimple-display="to"]',
+        );
         expect(left.value).toBe('0');
         expect(right.value).toBe('100');
+        expect(left.min).toBe('0');
+        expect(left.max).toBe('100');
+        expect(left.step).toBe('1');
+        expect(displayFrom.textContent).toBe('0');
+        expect(displayTo.textContent).toBe('100');
         expect(instance.sliderMin).toBe(0);
         expect(instance.sliderMax).toBe(100);
     });
@@ -94,18 +105,57 @@ describe('RangeSliderSimple', () => {
             '[wt-rangeslidersimple-element="slider-wrapper"]',
         );
         const instance = new RangeSliderSimple(wrapper);
+
+        instance.setTo('5');
+        instance.setFrom('4.9');
+
+        // min(4.9, 5 - 0.3) = 4.7 — parseInt would wrongly yield 4
+        expect(
+            wrapper.querySelector('[wt-rangeslidersimple-element="input-left"]')
+                .value,
+        ).toBe('4.7');
+        expect(
+            wrapper.querySelector('[wt-rangeslidersimple-element="input-right"]')
+                .value,
+        ).toBe('5');
+    });
+
+    test('constrainRightValue keeps at least minDifference above left handle', () => {
+        mountSlider();
+        const wrapper = document.querySelector(
+            '[wt-rangeslidersimple-element="slider-wrapper"]',
+        );
+        const instance = new RangeSliderSimple(wrapper);
         const left = wrapper.querySelector(
             '[wt-rangeslidersimple-element="input-left"]',
         );
         const right = wrapper.querySelector(
             '[wt-rangeslidersimple-element="input-right"]',
         );
 
-        instance.setTo('5');
-        instance.setFrom('4.9');
+        instance.setFrom('60');
+        instance.setTo('30');
 
-        // min(4.9, 5 - 0.3) = 4.7 — parseInt would wrongly yield 4
-        expect(left.value).toBe('4.7');
-        expect(right.value).toBe('5');
+        // max(30, 60 + 1) = 61 with mindifference === 1
+        expect(left.value).toBe('60');
+        expect(right.value).toBe('61');
+    });
+
+    test('reset restores min and max on both inputs', () => {
+        mountSlider();
+        const wrapper = document.querySelector(
+            '[wt-rangeslidersimple-element="slider-wrapper"]',
+        );
+        const instance = new RangeSliderSimple(wrapper);
+        instance.setRange('40', '50');
+        instance.reset();
+        const left = wrapper.querySelector(
+            '[wt-rangeslidersimple-element="input-left"]',
+        );
+        const right = wrapper.querySelector(
+            '[wt-rangeslidersimple-element="input-right"]',
+        );
+        expect(left.value).toBe('0');
+        expect(right.value).toBe('100');
     });
 });

docs/Functional/RangeSlider.md

@@ -30,10 +30,14 @@ Add the script to your project and include the required attributes and elements
 
 Self-contained: **one** script tag. Use [jsDelivr](https://www.jsdelivr.com/) (not raw GitHub URLs) so the MIME type is JavaScript.
 
+Pin to a **Git tag** (not `@main`) so embeds don’t change when the default branch moves. This module is **1.1.0** above—use that release ref when it exists:
+
 ```html
-<script src="https://cdn.jsdelivr.net/gh/TheCodeRaccoons/WebTricks@main/Dist/Functional/RangeSlider.js"></script>
+<script src="https://cdn.jsdelivr.net/gh/TheCodeRaccoons/WebTricks@v1.1.0/Dist/Functional/RangeSlider.js"></script>
 ```
 
+If `v1.1.0` is not published on the repo yet, use another tag from [Releases](https://github.com/TheCodeRaccoons/WebTricks/releases), a **commit SHA**, or `@1` (major ref; same idea as the main README) until you ship a matching tag.
+
 ### Required Attributes
 
 - `wt-rangeslider-element="slider-wrapper"` - Container element

docs/Functional/RangeSliderSimple.md

@@ -26,9 +26,11 @@ The script is **self-contained** (single file, one `<script>` tag).
 One script tag. Use [jsDelivr](https://www.jsdelivr.com/) (not raw GitHub URLs) so the MIME type is JavaScript.
 
 ```html
-<script src="https://cdn.jsdelivr.net/gh/TheCodeRaccoons/WebTricks@main/Dist/Functional/RangeSliderSimple.js"></script>
+<script src="https://cdn.jsdelivr.net/gh/TheCodeRaccoons/WebTricks@1/Dist/Functional/RangeSliderSimple.js"></script>
 ```
 
+Pin with a [GitHub release](https://github.com/TheCodeRaccoons/WebTricks/releases) tag or commit hash instead of `@main` when you need a fixed build.
+
 ## Required attributes
 
 - `wt-rangeslidersimple-element="slider-wrapper"` — outer container