Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Waveform Hotcue Markers are shifted to the right #9917

Closed
mixxxbot opened this issue Aug 23, 2022 · 8 comments
Closed

Waveform Hotcue Markers are shifted to the right #9917

mixxxbot opened this issue Aug 23, 2022 · 8 comments
Labels
Milestone

Comments

@mixxxbot
Copy link
Collaborator

Reported by: Holzhaus
Date: 2020-04-05T15:55:08Z
Status: Fix Released
Importance: High
Launchpad Issue: lp1870942
Attachments: [Cue Markers Offset GIF](https://bugs.launchpad.net/bugs/1870942/+attachment/5347150/+files/Cue Markers Offset GIF), [Screenshot from 2020-04-10 18-48-46.png](https://bugs.launchpad.net/bugs/1870942/+attachment/5351646/+files/Screenshot from 2020-04-10 18-48-46.png)


The hotcue markers on the scrolling waveforms are slightly shifted to the right. I assume this is related to the drawing of the markers (so the cues are not actually shifted), because the shift offset stays the same when zooming in/out.

I attached a GIF that shows the problem.

@mixxxbot mixxxbot added the bug label Aug 23, 2022
@mixxxbot
Copy link
Collaborator Author

Commented by: Holzhaus
Date: 2020-04-05T15:55:08Z
Attachments: [Cue Markers Offset GIF](https://bugs.launchpad.net/mixxx/+bug/1870942/+attachment/5347150/+files/Cue Markers Offset GIF)

@mixxxbot
Copy link
Collaborator Author

Commented by: Holzhaus
Date: 2020-04-05T16:10:55Z


The same happens with the main cue marker. The intro/outro markers are not affected.

@mixxxbot
Copy link
Collaborator Author

Commented by: Be-ing
Date: 2020-04-10T20:34:39Z


I checked 2.2 and could not replicate this, so this is a regression introduced in master.

@mixxxbot
Copy link
Collaborator Author

Commented by: Be-ing
Date: 2020-04-10T21:31:53Z


My first guess was that #2238 was when the regression was introduced, but I tested the last commit of that branch (05d4464) and could not reproduce this bug.

@mixxxbot
Copy link
Collaborator Author

Commented by: Be-ing
Date: 2020-04-10T23:20:58Z


Git bisect identified f024e75 as the first bad commit.

@mixxxbot
Copy link
Collaborator Author

Commented by: Be-ing
Date: 2020-04-10T23:49:27Z
Attachments: [Screenshot from 2020-04-10 18-48-46.png](https://bugs.launchpad.net/mixxx/+bug/1870942/+attachment/5351646/+files/Screenshot from 2020-04-10 18-48-46.png)


Only cues without labels are affected

@mixxxbot
Copy link
Collaborator Author

Commented by: Be-ing
Date: 2020-04-11T00:09:35Z


Fix is here: #2648

@mixxxbot
Copy link
Collaborator Author

Issue closed with status Fix Released.

@mixxxbot mixxxbot transferred this issue from another repository Aug 24, 2022
@mixxxbot mixxxbot added this to the 2.3.0 milestone Aug 24, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

1 participant