Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Traktor S3: Fix issues with sampler and hotcue buttons #4676

Merged
merged 1 commit into from
Jul 21, 2022

Conversation

ywwg
Copy link
Member

@ywwg ywwg commented Feb 12, 2022

  • Properly re-light buttons on track load
  • Fix hotcues not changing color when color is updated
  • Fix sampler buttons not changing color on play
  • Default to 8, not 16 samplers (deck 2 sampler buttons duplicate samplers 1-8)

* Properly re-light buttons on track load
* Fix hotcues not changing color when color is updated
* Fix sampler buttons not changing color on play
* Default to 8, not 16 samplers (deck 2 sampler buttons duplicate samplers 1-8)
@Holzhaus Holzhaus added changelog This PR should be included in the changelog minor bug labels Feb 23, 2022
@Holzhaus Holzhaus added this to the 2.3.3 milestone Feb 23, 2022
@Holzhaus
Copy link
Member

Default to 8, not 16 samplers (deck 2 sampler buttons duplicate samplers 1-8)

Can you open a PR to update the manual please? At least the "Button Pads" section still describes the old behavior. Then we can merge.

@ywwg
Copy link
Member Author

ywwg commented Feb 24, 2022

will do. thanks for the review

@ywwg
Copy link
Member Author

ywwg commented Feb 24, 2022

mixxxdj/manual#474

@ronso0
Copy link
Member

ronso0 commented Apr 19, 2022

Is there anythng left to do here?

  • Default to 8, not 16 samplers (deck 2 sampler buttons duplicate samplers 1-8)

@ywwg just curious, what's the reason to change this?

@ywwg
Copy link
Member Author

ywwg commented Apr 21, 2022

@ywwg just curious, what's the reason to change this?

I was getting annoyed during a performance. I wanted the hotcues on one track, but still wanted access to the samplers. With the default set at 16, you can't use the hotcues on deck 1 and also access samples 1-8

@ywwg
Copy link
Member Author

ywwg commented Apr 21, 2022

I believe there is nothing left to do one this

@github-actions
Copy link

This PR is marked as stale because it has been open 90 days with no activity.

@github-actions github-actions bot added the stale Stale issues that haven't been updated for a long time. label Jul 21, 2022
@Swiftb0y Swiftb0y merged commit b06dc6a into mixxxdj:2.3 Jul 21, 2022
@Swiftb0y
Copy link
Member

Sorry this fell under the table.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
changelog This PR should be included in the changelog controller mappings minor bug stale Stale issues that haven't been updated for a long time.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants