mixxx
mixxx copied to clipboard
Delete Files: move files to trash instead of deleting instantly
Reported by: stefan-franz Date: 2022-06-27T17:09:54Z Status: Confirmed Importance: Low Launchpad Issue: lp1980001 Attachments: [Bildschirmfoto vom 2022-06-28 06-22-40.jpg](https://bugs.launchpad.net/bugs/1980001/+attachment/5600085/+files/Bildschirmfoto vom 2022-06-28 06-22-40.jpg)
Linux Mint 20.3 Cinnamon
After rightclick in the browser list / delete track files from list
- The track is deleted without trash (is it possible to send it to the trash?)
- A small window appears that 1 song is removed from the database - but the windows does not close itsself.
Commented by: stefan-franz Date: 2022-06-27T17:15:02Z
hmmm....now, after deleting 3 files more the windows close istsself.... I report if the error comes again.
Commented by: ronso0 Date: 2022-06-27T21:02:53Z
The "N track files were removed" dialog is not supposed to close itself. Are you saying the dialog pops up, and while you do nothing (no button on keyboard or controller pressed, no mouse click), it is closed magically?
Note that controller with a Browse or Trax push/rotary encoder have [Library],GoToItem mapped. Per context this will trigger different actions, see https://manual.mixxx.org/2.4/en/chapters/appendix/mixxx_controls.html#control-[Library]-GoToItem, e.g. in dialogs it can press buttons.
Commented by: ronso0 Date: 2022-06-27T21:06:54Z
- The track is deleted without trash (is it possible to send it to the trash?)
Yes, this is noted as TODO. It's just someone needs to implement it :)
Commented by: ronso0 Date: 2022-06-27T21:41:52Z
QFile::moveToTrash is only supported in Qt 5.15 and newer.
IIRC (someone correct me if I'm wrong) we wait for the first point release of the current Ubuntu LTS (22.04.1) in order to drop support for Ubuntu 20.04 which is the only operating system that requires supporting Qt 5.12, and use Qt 5.15+ https://wiki.ubuntu.com/Releases#Future
We could as well add it now with Qt version switches, but then we wouldn't have a consistent implementation on all supported OS :|
Commented by: stefan-franz Date: 2022-06-28T04:34:02Z Attachments: [Bildschirmfoto vom 2022-06-28 06-22-40.jpg](https://bugs.launchpad.net/mixxx/+bug/1980001/+attachment/5600085/+files/Bildschirmfoto vom 2022-06-28 06-22-40.jpg)
The small window in the front comes not always if i delete a track. If it comes, it stands as here on the screenshot and the progress bar does not work. The cancel Button works and the file is deletet from the database. I think only the gui does sometimes not wirk right - the database correction works.
Commented by: ronso0 Date: 2022-06-28T09:19:59Z
I thought you were referring to the Success dialog
To clarify, the workflow has multiple dialogs:
- if you open the menu from a deck and click Delete while the track is still being played, Mixxx will ask you to stop the deck first
- confirmation "Permanently delete these files from disk?"
- progress dialog, only shown if required. quoted from source code: /// This dialog /// only appears if processing takes longer than the given grace /// period. This avoids that an open context menu gets closed /// while processing only a few tracks.
- Success: "Track File(s) Deleted"
- Failure report if file(s) could not be deleted
Commented by: ronso0 Date: 2022-06-28T09:22:43Z
I hope you didn't discover any other issues with the workflow.
This bug now only covers moveToTrash. Please file a separate report for dialog issues so we can track those independently. Thank you for testing btw!
Commented by: daschuer Date: 2022-06-28T13:50:33Z
The limiting OS is macOs It required Qt 5.12 because of the waveforms. This will only be solved if one writes new waveforms, so we are not talking about a few month to wait.
It feels bad to hold this useful feature back for all users just for sake of consistent behavior. I would welcome a solution with a version guard.
Commented by: stefan-franz Date: 2022-07-09T10:52:08Z
Linux Mint 20.3 Cinnamon - Mixxx 2.4 alpha2 - 1225 Deleting Files works now without an hanging window as - point 2 of my original report.
Commented by: ronso0 Date: 2022-07-10T17:49:39Z
yes, that was a separate bug resolved by https://github.com/mixxxdj/mixxx/pull/4831
Fixed for builds using Qt => 5.15 by #11842