-
Notifications
You must be signed in to change notification settings - Fork 279
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
feat: error and warning markings in IDE editor #1513
Conversation
± Registry diff
📊 PerformanceKeyNote that each bar component rounds up to the nearest 100ms, so each full bar is an overestimate by up to 400ms.
If a row has only one bar instead of four, that means it's not a trio and the bar just shows the total time spent for that example, again rounded up to the nearest 100ms. Data
|
Deploying with Cloudflare Pages
|
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! Thanks for getting to this issue finally. Testing this on the preview deployment felt great 💯
@@ -608,6 +612,246 @@ canvas { | |||
} | |||
}; | |||
|
|||
export const errLocs = ( |
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.
Another way would be to add this to an umbrella type so every error is required to identify the offending source location. This is also fine, but the downside is yet another place to update when a new error is added.
range: SourceRange; | ||
}; | ||
|
||
export const toErrorLoc = (node: { |
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.
Is this function only renaming and grouping some values? Looks like you had to do some work when feeding them to monaco anyway. Why is this step necessary?
Description
Since many types of warnings and errors include source location information, we can utilize these information to mark them in the IDE editor.
Implementation strategy and design decisions
errLocs
, that extracts the location information (which file, which row, which column) from each error and warning, exceptStyleErrorList
.StyleErrorList
essentially stores multiple Style errors, so we handle them specially.Checklist