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

improve readabality #430 add standby notice #613

Open
wants to merge 6 commits into
base: main
Choose a base branch
from

Conversation

pi43r
Copy link

@pi43r pi43r commented Feb 12, 2024

I couldn't figure out how to add line breaks either. But this should be slightly better.
Also added a notice that the controller only starts working when set as output device (after which one can remove it, and it still works as a midi controller)

edit(ronso0) based on #430

@@ -26,6 +26,7 @@ Mapping
Korg Kaoss DJ (schematic view)


<<<<<<< Updated upstream
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There's some residue from conflict resolution

@pi43r
Copy link
Author

pi43r commented Feb 12, 2024

fixed the unexpected merge error.
I left some functions inside as "not used yet" or "not implemented yet" in the hopes that someone (maybe me if I learn a bit mor) will pick it up in the future

@pi43r
Copy link
Author

pi43r commented Feb 14, 2024

FX button based on mixxxdj/mixxx#12807
Also figured out how to do line breaks

Looks good to me now

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.

3 participants