-
Notifications
You must be signed in to change notification settings - Fork 28
rest shifting according to clarified spec #90
Conversation
@@ -647,14 +647,18 @@ function FlatEditor({ | |||
setRefId(score.scoreId); | |||
}) | |||
.catch((e) => { | |||
e.message = `flat error: ${e.message}, not loaded from scoreId, score: ${JSON.stringify(score)}, orig: ${orig}, colors: ${colors}`; | |||
if (debugMsg){ |
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.
ok, but like, can you still keep debugMsg
when it's available at least in the console error. the point of the hasty change was to be able to distinguish which instances of the Flateditor are experiencing the errors?
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.
Admittedly, my commenting out was hasty, especially considering the latter of the runtime errors seems to only happen when the scores lock. uncommented now
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.
Can you throw debugMsg
somewhere when it's available?
per review request
only debugMsg and console.error
just debugMsg
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.
ship it!
This should take care of rests breaking the shifts.
notes will now have an
isRest
property if they are part of a measure passed through these functions. Makes life easier.