QT_SCREEN_SCALE_FACTORS causes CUE marker display misplacement

Bug #1917253 reported by Attila Schler
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Mixxx
Fix Released
Medium
ronso0

Bug Description

Setting QT_SCREEN_SCALE_FACTORS causes CUE marker display misplacement on waveforms

See on the screenshots in the attached pdf:
QT_SCREEN_SCALE_FACTORS_AND_CUE_MISPLACEMENT.PDF

Mixxx 2.3.0-beta (build 2.3 r8043)
Operating System: openSUSE Tumbleweed 20210220
Qt Version: 5.15.2
Kernel Version: 5.10.16-1-default
Graphics Processor: Mesa DRI Intel® HD Graphics 3000

Revision history for this message
Attila Schler (atskler) wrote :
Be (be.ing)
Changed in mixxx:
importance: Undecided → Medium
Revision history for this message
Attila Schler (atskler) wrote :

Should I re-report this bug under launchpad.net/mixxx/2.3 ?

Revision history for this message
Jan Holthuis (holthuis-jan) wrote :

No, I just changed the milestone.

Changed in mixxx:
milestone: none → 2.3.0
Revision history for this message
Owen Williams (ywwg) wrote :

Yeah I've seen this on windows

Revision history for this message
Attila Schler (atskler) wrote :

I would like to ask if this bug will be fixed for the 2.3 release?

Revision history for this message
ronso0 (ronso0) wrote : Re: QT_SCREEN_SCALE_FACTORS causes CUE label display misplacement

Very unlikely because we're about to release 2.3 atm

summary: - QT_SCREEN_SCALE_FACTORS causes CUE marker display misplacement
+ QT_SCREEN_SCALE_FACTORS causes CUE label display misplacement
Changed in mixxx:
status: New → Confirmed
milestone: 2.3.0 → 2.3.1
Revision history for this message
Attila Schler (atskler) wrote :

Important, so please note it is not just the CUE label, but the complete marker is misplaced on the display. If you jump to a cue, mix will jump to it and will play from it, but the marker will not show the correct position.

.

I would like to add that this bug – at least for me – is quite frustrating, because on my notebook with its standard resolution everything is too small for using mixxx on a live gig: checking the display from about 80-100 cm distance.

Surely I can work around this problem with decreasing the screen resolution.

I think this feature would be important for others, and this problem can impact and mislead others already using this feature - regarding the displayed and the actual cue position.

Revision history for this message
ronso0 (ronso0) wrote :

sorry, my bad, I was confuding the play position with the Cue marker.

summary: - QT_SCREEN_SCALE_FACTORS causes CUE label display misplacement
+ QT_SCREEN_SCALE_FACTORS causes CUE marker display misplacement
Revision history for this message
ronso0 (ronso0) wrote :

All waveform markers are affected.
Though there's a difference for text labels and those with icons:
TEXT:
 * text area is cropped
 * stroke is shifted right
  (the longer the text, the further it's shifted)
 * stroke is not painted top to bottom, but stops at
  (top-bottom)/scale factor, so for scale factor of 1.5 it's only
  the upper 2/3 that's painted
ICON:
 * entire icon is painted
 * stroke is shifted slightly (about half the icon width in Deere),
   though that differs for the CUE and loop_in icon

A wild guess: somwehere in the marker position calculation there's a rounding issues, and appearantly the scale factor is not considered correctly (incomplete stroke for text labels)

Revision history for this message
ronso0 (ronso0) wrote :

the marker text size is not scaled at all...

got it!

Changed in mixxx:
status: Confirmed → In Progress
assignee: nobody → ronso0 (ronso0)
milestone: 2.3.1 → 2.3.0
Revision history for this message
ronso0 (ronso0) wrote :
Revision history for this message
ronso0 (ronso0) wrote :

okay, that was easier than I though.
please test!

Revision history for this message
Attila Schler (atskler) wrote :

Thank you! I will test it once it is merged.

Revision history for this message
ronso0 (ronso0) wrote :

We prefer testing before merging changes ;)
For each PR there are test builds for Linux, Win and mac (artifacts).
see https://github.com/mixxxdj/mixxx/wiki/Testing for details, especially the backup part.

Revision history for this message
ronso0 (ronso0) wrote :

and the fix for pixmap icons
https://github.com/mixxxdj/mixxx/pull/3938

Changed in mixxx:
status: In Progress → Fix Committed
Revision history for this message
Attila Schler (atskler) wrote :

Thank You for fixing this issue for v2.3!

Changed in mixxx:
status: Fix Committed → Fix Released
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/10335

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.