Skip to content

Commit 1a31253

Browse files
committed
Merge pull request #22 from esseb/text-selection
Prevent text selection during drag
2 parents 75f90f8 + ce7b7e1 commit 1a31253

File tree

2 files changed

+16
-0
lines changed

2 files changed

+16
-0
lines changed

src/InputRange/InputRange.js

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -422,6 +422,8 @@ export default class InputRange extends React.Component {
422422
return;
423423
}
424424

425+
event.preventDefault();
426+
425427
const key = getKeyByPosition(this, position);
426428

427429
this.updatePosition(key, position);

test/InputRange.spec.js

Lines changed: 14 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -357,9 +357,23 @@ describe('InputRange', () => {
357357
event = {
358358
clientX: 100,
359359
clientY: 200,
360+
preventDefault: jasmine.createSpy('preventDefault'),
360361
};
361362
});
362363

364+
it('should call event.preventDefault if not disabled', () => {
365+
inputRange.handleTrackMouseDown(event, track, position);
366+
367+
expect(event.preventDefault).toHaveBeenCalledWith();
368+
});
369+
370+
it('should not call event.preventDefault if disabled', () => {
371+
inputRange = renderComponent(<InputRange disabled={true} defaultValue={10} onChange={onChange}/>);
372+
inputRange.handleTrackMouseDown(event, track, position);
373+
374+
expect(event.preventDefault).not.toHaveBeenCalledWith();
375+
});
376+
363377
it('should not set a new position if disabled', () => {
364378
inputRange = renderComponent(<InputRange disabled={true} defaultValue={10} onChange={onChange}/>);
365379
spyOn(inputRange, 'updatePosition');

0 commit comments

Comments
 (0)