Skip to content

Commit 0503b6e

Browse files
authored
[NOCI] Add touch events to range slider (#176)
* Add touch events to range slider * add test
1 parent 8b34a9f commit 0503b6e

File tree

2 files changed

+20
-0
lines changed

2 files changed

+20
-0
lines changed

spec/components/Filters/Filters.test.jsx

Lines changed: 18 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -419,5 +419,23 @@ describe('Testing Component: Filters', () => {
419419
const updatedFiltersWithMaxSliderMove = getRequestFilters(container);
420420
expect(updatedFiltersWithMaxSliderMove.price[0].indexOf('70')).not.toBe(-1);
421421
});
422+
423+
it('SliderRange: Using Touch Events - Upon moving the slider buttons, requestFilters should be updated', async () => {
424+
const { container } = render(<TestFiltersApplied />);
425+
const cioMinSlider = container.querySelector('.cio-min-slider');
426+
const cioMaxSlider = container.querySelector('.cio-max-slider');
427+
428+
fireEvent.change(cioMinSlider, { target: { value: 20 } });
429+
fireEvent.touchEnd(cioMinSlider);
430+
431+
const updatedFiltersWithMinSliderMove = getRequestFilters(container);
432+
expect(updatedFiltersWithMinSliderMove.price[0].indexOf('20')).not.toBe(-1);
433+
434+
fireEvent.change(cioMaxSlider, { target: { value: 70 } });
435+
fireEvent.touchEnd(cioMaxSlider);
436+
437+
const updatedFiltersWithMaxSliderMove = getRequestFilters(container);
438+
expect(updatedFiltersWithMaxSliderMove.price[0].indexOf('70')).not.toBe(-1);
439+
});
422440
});
423441
});

src/components/Filters/FilterRangeSlider.tsx

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -200,6 +200,7 @@ export default function FilterRangeSlider(props: FilterRangeSliderProps) {
200200
value={minValue}
201201
onChange={onMinSliderMove}
202202
onMouseUp={onSliderMoveEnd}
203+
onTouchEnd={onSliderMoveEnd}
203204
/>
204205
<input
205206
className='cio-max-slider'
@@ -210,6 +211,7 @@ export default function FilterRangeSlider(props: FilterRangeSliderProps) {
210211
value={maxValue}
211212
onChange={onMaxSliderMove}
212213
onMouseUp={onSliderMoveEnd}
214+
onTouchEnd={onSliderMoveEnd}
213215
/>
214216
</div>
215217
</div>

0 commit comments

Comments
 (0)