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
Implements block decorations feature. #152740
Conversation
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.
Looks good! I left a few small comments.
|
||
.monaco-editor .blockDecorations-block { | ||
position: absolute; | ||
content: " "; |
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.
why is this necessary?
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.
Right, this is not necessary - was a left over. Thanks!
this.domNode.appendChild(block); | ||
} | ||
const top = ctx.getVerticalOffsetForLineNumber(decoration.range.startLineNumber); | ||
const bottom = ctx.getVerticalOffsetForLineNumber(decoration.range.endLineNumber + 1); |
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.
This will contain any view zone immediately following this line. If you would like to limit the block strictly to the end line number, you would need to use something like ctx.getVerticalOffsetForLineNumber(decoration.range.endLineNumber) + lineHeight
. lineHeight
can be found in the editor options.
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.
Right, good catch. Unfortunately, this is the behavior I need, to cover deleted ranges in the 3wm editor.
I suggest to add a comment and merge it as this and solve the problem when someone does not want to include view zones.
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.
Nice!
No description provided.