Star ratings not reacting to click

Bug #1870909 reported by Foss-4
10
This bug affects 2 people
Affects Status Importance Assigned to Milestone
Mixxx
Status tracked in 2.4
2.3
Fix Released
Low
Unassigned
2.4
Fix Committed
Low
Unassigned

Bug Description

macOS 10.15.4
Mixxx 2.3.0-alpha-pre (build master r7219)
Skin: any

Setting a star rating is currently confusing and not at all behaving as expected:

On mouseover the stars are showing up, but mixxx is unresponsive on any click on one of the stars and it seems the rating is not recognized. See attached video.

Tags: gui
Revision history for this message
Foss-4 (foss-4) wrote :
tags: removed: rating star
Changed in mixxx:
status: New → Confirmed
importance: Undecided → Medium
Revision history for this message
Alex (al3xf) wrote :

I also find the rating behavior very confusing, actually on my workstation (macOS Catalina, Mixxx v2.2.3) if I mouse over the stars they show up regardless of the actual metadata and sometimes they stick to 5 stars after the mouse over.

I'd love to be able to use it as other metadata fields, i.e. set it directly from Mixxx, and not having any visual behavior change when mouse pointer goes over the stars.

Revision history for this message
Foss-4 (foss-4) wrote :

More odd behavior in regards to mouse behavior and star ratings on macOS:

when trying to set a star rating in the deck, it is misaligned and seems to react one star off. see screencast.

Revision history for this message
Foss-4 (foss-4) wrote :
Revision history for this message
ronso0 (ronso0) wrote :

the rating is applied on mouse release.
do you use a mouse or the mac trackpads?

Revision history for this message
ronso0 (ronso0) wrote :

> I'd love to be able to use it as other metadata fields, i.e. set it directly from Mixxx

@al3xf What do you mean?
There are controls to change the star rating via controller. In https://github.com/mixxxdj/mixxx/wiki/mixxxcontrols look for 'stars_up'. I guess you can use those controls for a keyboard mapping as well.

Revision history for this message
ronso0 (ronso0) wrote :

@al3xf Are you able to build mixxx on MacOS?
If so you can try to catch the mousePressEvent instead of mouseReleaseEvent.
Repsective code is there: https://github.com/mixxxdj/mixxx/blob/master/src/widget/wstarrating.cpp#L143

Revision history for this message
Foss-4 (foss-4) wrote :

"the rating is applied on mouse release."
using trackpad here: https://bugs.launchpad.net/mixxx/+bug/1870909/+attachment/5394963/+files/star%20ratings.mp4

Is it intended to adapt any change without an actual click? I would expect only to change something when clicking on it (not hoovering - and accidentally lifting the finger).

Revision history for this message
ronso0 (ronso0) wrote :

I'm not sure what the reason was to use the release event.
on Linux and Windows this works with a explicit clicks only.

alternatively we may use both mousePressEvent AND mouseRelaseEvent and in the latter check if both occured in roughly the same spot (custom single click detection).

Or we simply use mouseDoubleClickEvent on macOS only because the current implementation works on Linux and Windows.

btw: does the library star rating work or is it suffering the same issue?

Revision history for this message
Foss-4 (foss-4) wrote :

library ratings are not affected by the "off by one star" problem.

Revision history for this message
Foss-4 (foss-4) wrote :

Behavior in 2.4.0-alpha-pre (build master r7487) has improved the star rating behavior in library view a lot.

I think it's fair to close this issue. Setting to fix commited even without specific commit as there is no worksforme state.

Tested w macOS 10.15.6.

Followup issue for the misaligned star rating in deck view is here:
https://bugs.launchpad.net/mixxx/+bug/1890430

@Alex would be great if you could test this as well, confirm the followup bug and maybe also confirm that the problem with star ratings in library view is also solved for you.

Changed in mixxx:
status: Confirmed → Fix Committed
milestone: none → 2.4.0
Revision history for this message
Foss-4 (foss-4) wrote :

Tested and 2.3.0-beta (build 2.3 r7546) on macOS 10.15.6 + Supplement Update works as expected.

Revision history for this message
Swiftb0y (swiftb0y) wrote :

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

lock status: Metadata changes locked and limited to project staff
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.