Add option to show current track in default file manager

[jus]

We should add a option to the library`s right-click menue to locate a track in the filesystem that opens the OS`s default file manager with relevant folder and the current track marked.

This is especially useful to reveal recordings ( from inside the Recordings tab) and is usual in audio applications - simply a convenience feature.

[RJ Skerry-Ryan (rryan)]

nice idea :)

On Sun, Feb 26, 2012 at 7:25 AM, jus <email address hidden> wrote:

> Public bug reported:
>
> We should add a option to the library`s right-click menue to locate a
> track in the filesystem that opens the OS`s default file manager with
> relevant folder and the current track marked.
>
> This is especially useful to reveal recordings ( from inside the
> Recordings tab) and is usual in audio applications - simply a
> convenience feature.
>
> ** Affects: mixxx
> Importance: Wishlist
> Status: Confirmed
>
>
> ** Tags: usability wishlist
>
> ** Tags added: usability wishlist
>
> ** Changed in: mixxx
> Importance: Undecided => Wishlist
>
> ** Changed in: mixxx
> Status: New => Confirmed
>
> --
> You received this bug notification because you are a member of Mixxx
> Development Team, which is subscribed to Mixxx.
> https://bugs.launchpad.net/bugs/941379
>
> Title:
> Add option to show current track in default file manager
>
> To manage notifications about this bug go to:
> https://bugs.launchpad.net/mixxx/+bug/941379/+subscriptions
>

[mutil (mutil)]

A patch which adds this option.
Works with multiple files and opens as many browsers as the different directories of files that are selected.
We should discuss if we want this behavior(inspired by clementine's way) or we should limit the open browser option to only one selected file.

[RJ Skerry-Ryan (rryan)]

Hey mutil,

Short and sweet :). Looks good.

Is there a reason why you do:

QUrl url = QUrl::fromLocalFile(trackModel->getTrackLocation(index));
if (!url.isValid()) {
...
}

if (url.scheme() != "file")
  continue;

QString path = url.toLocalFile();

instead of just:

QString path = trackModel->getTrackLocation(index);

I figure you just copy/pasted from other parts of wtracktableview.cpp that first created a QUrl but I think it's fine to just do the latter. Just asking in case you found some case where it was necessary to do that.

[mutil (mutil)]

Hey Ryan,
You are right, I just copied this part from mouseMoveEvent, no real reason.
Your modified commit works just fine.

[Swiftb0y (swiftb0y)]

Mixxx now uses GitHub for bug tracking. This bug has been migrated to:
https://github.com/mixxxdj/mixxx/issues/6288