-
Notifications
You must be signed in to change notification settings - Fork 2.6k
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
Rework anchor lines drawing #5738
Conversation
The test failure is related to the issue #301436 which is already present in master, so it is better to fix that issue rather than this particular crash on a test scenario. |
0b58f53
to
55a3600
Compare
30365bb
to
45488bf
Compare
c2e06ae
to
a68c899
Compare
a68c899
to
91431b7
Compare
I reckon the "work in progress" label should go? |
@@ -75,10 +75,14 @@ void ScoreView::doDragElement(QMouseEvent* ev) | |||
mscore->play(e); | |||
_score->setPlayNote(false); | |||
} | |||
QLineF anchor = e->dragAnchor(); | |||
QVector<QLineF> anchorLines = e->dragAnchorLines(); | |||
const QPointF pageOffset(e->findAncestor(ElementType::PAGE)->pos()); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think this might be where we are getting the extra page offset that is causing https://musescore.org/en/node/306096. If I comment out the translate that happens below, it fixes the case at hand, but I'm not totally sure it doesn't break something else.
This is a part of redesign of elements dragging system which I got split off the
anchors_rebasing
branch to make the amount of changes more manageable. This PR is dedicated to anchor lines drawing redesign and includes:Possibly some more changes could be added in future.