Skip to content

Option to adjust playback speed (tempo) via middle gesture on main player screen#200

Merged
polymorphicshade merged 2 commits intopolymorphicshade:devfrom
klaviartur:feature_tempo_gesture
May 10, 2025
Merged

Option to adjust playback speed (tempo) via middle gesture on main player screen#200
polymorphicshade merged 2 commits intopolymorphicshade:devfrom
klaviartur:feature_tempo_gesture

Conversation

@klaviartur
Copy link
Copy Markdown
Contributor

@klaviartur klaviartur commented Feb 8, 2025

Add third gesture area in the middle third of player screen
Add third gesture option: tempo / playback speed
Add settings item for middle gesture

Already working (for 3 weeks) and beloved (by some people) feature in MaintainTeam/LastPipeBender

asandikci

This comment was marked as resolved.

@asandikci
Copy link
Copy Markdown
Collaborator

Apart from these I can verify that this feature works properly (in my fork)

@klaviartur
Copy link
Copy Markdown
Contributor Author

I'm sorry, I was so careless. This was done in a hurry and I naively hoped, I can just copy whole files.

How do we proceed? Shall I cancel this PR and open new one only with intended changes?

@asandikci
Copy link
Copy Markdown
Collaborator

It depends on you. You can revert changes manually and force push too.

@klaviartur klaviartur force-pushed the feature_tempo_gesture branch 2 times, most recently from 47d01a4 to 26c2a45 Compare February 8, 2025 21:29
@klaviartur klaviartur force-pushed the feature_tempo_gesture branch from 26c2a45 to 6e6e6bf Compare February 8, 2025 21:30
@klaviartur
Copy link
Copy Markdown
Contributor Author

Okay, a lot of trial and error, but seems done now. Only intended changes should appear.

@asandikci asandikci changed the base branch from master to dev February 9, 2025 15:22
@abuturabofficial

This comment was marked as off-topic.

@asandikci

This comment was marked as off-topic.

@polymorphicshade polymorphicshade merged commit 2792426 into polymorphicshade:dev May 10, 2025
1 check failed
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.

4 participants