Skip to content

fix(media): click-to-seek on position slider and volume slider in audio widget #1493#1539

Open
Mestane wants to merge 1 commit into
caelestia-dots:mainfrom
Mestane:fix/media-slider-click-seek
Open

fix(media): click-to-seek on position slider and volume slider in audio widget #1493#1539
Mestane wants to merge 1 commit into
caelestia-dots:mainfrom
Mestane:fix/media-slider-click-seek

Conversation

@Mestane

@Mestane Mestane commented Jun 9, 2026

Copy link
Copy Markdown
Contributor

Problem

Clicking directly on the media position slider had no effect — the playback position did not change. Only dragging worked.

In StyledSlider.qml, onReleased was using posBinding.value which evaluates to pressStartPos + dragMovement. Since dragMovement is only updated in onPositionChanged (which never fires on a click with no movement), it remained 0 on release — causing the interaction to emit the original position instead of the clicked position.

Fix

In onReleased, check whether any drag movement occurred. If not, calculate the position directly from the release event's x coordinate instead of relying on posBinding.

Notes

This also resolves the snap-back issue described in #1485 for click interactions, as the correct position is now emitted on the first try.

Video

recording_20260609_13-50-55.mp4

fixed( nexus ):#1493

@swopnil7

swopnil7 commented Jun 9, 2026

Copy link
Copy Markdown

Clicking directly on the media position slider had no effect — the playback position did not change. Only dragging worked.

Also the same case for volume slider in audio widget.

@Mestane

Mestane commented Jun 9, 2026

Copy link
Copy Markdown
Contributor Author

Clicking directly on the media position slider had no effect — the playback position did not change. Only dragging worked.

Also the same case for volume slider in audio widget.

Yes, this PR fixes both the same root cause applies to any StyledSlider instance where interactionOnMove: false. The volume slider in the audio widget should also work with direct clicks after this.
It works for me.

Video

recording-20260609-143347.mp4

@Mestane Mestane force-pushed the fix/media-slider-click-seek branch from 53594ad to a56e74c Compare June 10, 2026 10:16
@Mestane Mestane changed the title fix(media): click-to-seek on position slider #1493 fix(media): click-to-seek on position slider and volume slider in audio widget #1493 Jun 10, 2026
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