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

Explore showing multiline decorations #181

Open
1 of 8 tasks
usernamehw opened this issue Aug 20, 2023 · 1 comment
Open
1 of 8 tasks

Explore showing multiline decorations #181

usernamehw opened this issue Aug 20, 2023 · 1 comment
Assignees

Comments

@usernamehw
Copy link
Owner

usernamehw commented Aug 20, 2023

Try finding a space to fit as much text as possible (Tetris).

Whether it's a very long single line problem, or a multiline problem => find the best empty box to fit it (using multiple single line decorations).

multiline_demo


  • Highlight line where problem is from (since it's rendered in random position in the viewport)
  • Closest problem multiline; doesn't calculate the best position properly atm
  • Closest problem multiline by severity (error before warning)
  • Closest problem multiline in viewport
  • Align multiline message
  • Split single-line long error into multiple lines
  • Option to show multiple multiline error or an option to render as usual (all errors) if there are no multiline/long errors
  • Show multiple problems from 1 line

How to enable and configure experimental version:

"errorLens.followCursor": "closestProblemMultiline",
"errorLens.multilineMessage": {
	"decorationMaxNumberOfLines": 4,
	"preferFittingMessageMultiplier": 2,
	"highlightProblemLine": "line",
	"alignStart": 0,
	"alignEnd": 0,
	"margin": 4,
	"padding": 1,
	"borderRadius": "",
	"useFixedPosition": true,
},
@usernamehw usernamehw self-assigned this Aug 20, 2023
usernamehw added a commit that referenced this issue Oct 20, 2023
align multiline message in the future #181
@goyalyashpal
Copy link

goyalyashpal commented Nov 3, 2023

hi! thanks for such a wonderful extension and including this on open-vsx too.

is this issue just about showing single but long diagnostic line or would also cover multiple diagnostic messages for same line??

e.g.:

Syntax error, insert "VariableDeclarators" to complete LocalVariableDeclaration [21:17]
Syntax error, insert ";" to complete LocalVariableDeclarationStatement [21:17]

image

image

image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants