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

View menu: add 'Show Auto DJ' action #12890

Draft
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

ronso0
Copy link
Member

@ronso0 ronso0 commented Feb 28, 2024

requested here https://mixxx.zulipchat.com/#narrow/stream/109122-general/topic/AutoDJ.20as.20a.20View.20option

AutoDJ is a special feature that may need immediate attention, i.e. a way to activate it quickly, that's why I think a shortcut seems reasonable.
Screenshot_2024-02-28_13-21-01

Which keyboard shortcut?

There are already Ctrl+1..6
Ctrl+7 would be next, but maybe we want to extend the skin controls, and then it would be better to have consecutive shortcuts for toggles ("Show AutoDJ is a trigger).
What about Ctrl+9?

@fwcd
Copy link
Member

fwcd commented Feb 28, 2024

Maybe adding the other "main" sections would make sense too, e.g. "Show Tracks"?

I'd probably lean towards a non-numeric shortcut so we can reserve those for skin controls and go for a letter (e.g. Ctrl + Alt or Shift + A?)

@ronso0 ronso0 force-pushed the autodj-show-menu-action branch from c63da3c to 778c5e8 Compare February 29, 2024 00:29
@ronso0
Copy link
Member Author

ronso0 commented Feb 29, 2024

have to reconsider this:
IIUC the idea was to quickly jump to AutoDJ when you're way down in the sidebar.
Currently this action selects AutoDJ but also scrolls there.

However, since you (maybe) only want to activate AutoDJ and then continue working in the last view (computer, history, ...), it would be helpful if the position of the sidebar were fixed. Sure, the current view would be deselected, but it would still be easy to pick up where you left off.
With Tracks or other views it could be confusing, but the AutoDJ view is recognizable.

What do you think?

Maybe adding the other "main" sections would make sense too, e.g. "Show Tracks"?

And we'd have #9912 (somehow) : )

&QAction::triggered,
this,
&AutoDJFeature::slotRemoveCrateFromAutoDj);
qRegisterMetaType<AutoDJProcessor::AutoDJState>("AutoDJState");
Copy link
Member

Choose a reason for hiding this comment

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

This extra indentation looks wrong. A clang-format bug? Can we workaround it?

Copy link
Member Author

Choose a reason for hiding this comment

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

True. If I revert that clang-format insists on 8 spaces.
No idea how to work around it.

Copy link
Member Author

Choose a reason for hiding this comment

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

Revert and ignore clang-format?

Copy link
Member

Choose a reason for hiding this comment

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

Yes, I would have no issue to merge it under clang-format complains. (It should help us, not annoy us)

Copy link
Member Author

Choose a reason for hiding this comment

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

reverted

@daschuer
Copy link
Member

Works already :-)

@daschuer
Copy link
Member

However, since you (maybe) only want to activate AutoDJ and then continue working in the last view (computer, history, ...), it

Correct. I played a bit with it and it reminds me to taped browsing. Search for something in the Library, verify the Auto-DJ playlist.
Continue searching.

We may emulate that by adding tabs to the sidebar ... Or a back button.

The ultimate solution "Open Auto DJ in a new Window" We have discussed the idea earlier to have a skin with two windows.

Or even allow to have two or more tabs on the library. This will allow to switch between any two contexts without headaches.

@daschuer
Copy link
Member

I get hocked to the tapped library idea. Can we implement it with the singleton library feature in two tabs? In a first version each tap "just" need to restore the stored library table. Caveat: We cant have two tabs with the same playlist open and a saved state for each.

@ronso0
Copy link
Member Author

ronso0 commented Feb 29, 2024

Yes, these are nice ideas and part of the library redesign discussion.
I'm not going ot work on it : )

Here, I think we are good if we simply add something like SidebarModel::selectFeatureNoScroll(LibraryFeature* pFeature). Though this will inevtably be mostly redundant with the existing selectFeature, hmm.

@vespadj
Copy link

vespadj commented Mar 13, 2024

My suggestion
AutoDJ in Tray bar

@ronso0
Copy link
Member Author

ronso0 commented Mar 13, 2024

Thank you, having a (additional*) topbar toggle is a good idea!
*Shade skin has no top bar, so we should always provide the action in the View menu

Note: moving widgets out of the top bar is out of scope here.
There is WIP regarding a global status bar though #3406

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 Jun 12, 2024
@ronso0 ronso0 force-pushed the autodj-show-menu-action branch from 778c5e8 to dfdbd59 Compare July 29, 2024 11:20
@github-actions github-actions bot removed the stale Stale issues that haven't been updated for a long time. label Jul 30, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants