open cover art view after single-click release

[mixxxbot]

Reported by: ronso0
Date: 2019-09-19T21:15:09Z
Status: Fix Released
Importance: Low
Launchpad Issue: lp1844730
Tags: coverart, usability

---

Like the track title and artist widgets, a deck cover art can be dragged to other decks, samplers or somewhere in the library.
Currently the cover art window opens on mouse button PRESS which partially covers the GUI and thereby blocks certain areas for drag moves.

open cover art view not before single-click is released to keep the view clean for drag movements.
Usual longpress timeout should be apllied.

It could also open if the pointer is still above same widget when being released, but that would still/again be the case when when aborting a drag move. Opening the cover art wouldn't be helpful then.

[mixxxbot]

Commented by: ronso0
Date: 2019-09-22T15:20:54Z

---

quick solution:
* in mousePressEvent() in wcoverart.cpp
  if (event->button() == Qt::LeftButton) {
    start longpress timer;}
* add mouseReleaseEvent() {
  if (event->button() == Qt::LeftButton && timer not expired yet) {
    stop timer, delete timer;
    open cover art window;}}

[mixxxbot]

Commented by: ronso0
Date: 2019-09-23T20:22:06Z

---

#2296

[mixxxbot]

Commented by: daschuer
Date: 2019-09-24T06:19:09Z

---

What is the issue you are trying to solve here?

[mixxxbot]

Commented by: ronso0
Date: 2019-09-24T13:30:57Z

---

description updated, sorry for not being clear.

Currently the cover art window opens on mouse button PRESS which partially covers the GUI and thereby blocks certain areas for cover art drag moves.

[mixxxbot]

Issue closed with status Fix Released.