-
-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
(fix) Library: repair Del hotkey in sidebar, add Clear Queue action to AutoDJ, add hotkeys #13364
Conversation
762e243
to
a87712f
Compare
ding dong We're slowly approaching the 2.5 release so wdyt about this?
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Oh, just realized that I had a pending minor review comment. Here is it.
Sorry for not being very responsive, I am travelling. LGTM but not merging as there is a pre-commit issue. |
91c13c0
to
a377746
Compare
Thanks.
I guess that was from the tr code suggestions, fixed by merging that commit as fixup. |
This fits in nicely with the "Clear Auto DJ Queue" action introduced by mixxxdj#13364.
This fits in nicely with the "Clear Auto DJ Queue" action introduced by mixxxdj#13364.
Fixes #13361
Alternatively, we pick only the fix for 2.5 and the new action goes to 2.6