diff --git a/packages/webawesome/docs/docs/resources/changelog.md b/packages/webawesome/docs/docs/resources/changelog.md index f6fb18584..e4e2c3847 100644 --- a/packages/webawesome/docs/docs/resources/changelog.md +++ b/packages/webawesome/docs/docs/resources/changelog.md @@ -10,6 +10,10 @@ Web Awesome follows [Semantic Versioning](https://semver.org/). Breaking changes Components with the Experimental badge should not be used in production. They are made available as release candidates for development and testing purposes. As such, changes to experimental components will not be subject to semantic versioning. +## Next + +- Fixed a bug in `` that caused some touch devices to end up with the incorrect value [issue:1703] + ## 3.0.0 - 🚨 BREAKING: Changed `appearance="filled outlined"` to `appearance="filled-outlined"` in the following elements [issue:1127] @@ -492,4 +496,4 @@ Many of these changes and improvements were the direct result of feedback from u -Did we miss something? [Let us know!](https://github.com/shoelace-style/webawesome/discussions) \ No newline at end of file +Did we miss something? [Let us know!](https://github.com/shoelace-style/webawesome/discussions) diff --git a/packages/webawesome/src/internal/drag.ts b/packages/webawesome/src/internal/drag.ts index a1fe23e4b..10c249884 100644 --- a/packages/webawesome/src/internal/drag.ts +++ b/packages/webawesome/src/internal/drag.ts @@ -84,8 +84,8 @@ export class DraggableElement { } private handleDragStart = (event: PointerEvent | TouchEvent) => { - const clientX = supportsTouch && 'touches' in event ? event.touches[0].clientX : (event as PointerEvent).clientX; - const clientY = supportsTouch && 'touches' in event ? event.touches[0].clientY : (event as PointerEvent).clientY; + const clientX = 'touches' in event ? event.touches[0].clientX : (event as PointerEvent).clientX; + const clientY = 'touches' in event ? event.touches[0].clientY : (event as PointerEvent).clientY; if ( this.isDragging || @@ -105,8 +105,8 @@ export class DraggableElement { }; private handleDragStop = (event: PointerEvent | TouchEvent) => { - const clientX = supportsTouch && 'touches' in event ? event.touches[0].clientX : (event as PointerEvent).clientX; - const clientY = supportsTouch && 'touches' in event ? event.touches[0].clientY : (event as PointerEvent).clientY; + const clientX = 'touches' in event ? event.touches[0].clientX : (event as PointerEvent).clientX; + const clientY = 'touches' in event ? event.touches[0].clientY : (event as PointerEvent).clientY; this.isDragging = false; document.removeEventListener('pointerup', this.handleDragStop); @@ -117,8 +117,8 @@ export class DraggableElement { }; private handleDragMove = (event: PointerEvent | TouchEvent) => { - const clientX = supportsTouch && 'touches' in event ? event.touches[0].clientX : (event as PointerEvent).clientX; - const clientY = supportsTouch && 'touches' in event ? event.touches[0].clientY : (event as PointerEvent).clientY; + const clientX = 'touches' in event ? event.touches[0].clientX : (event as PointerEvent).clientX; + const clientY = 'touches' in event ? event.touches[0].clientY : (event as PointerEvent).clientY; // Prevent text selection while dragging window.getSelection()?.removeAllRanges();