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

Hide or shorten error codes in problems list #233

Closed
bwegge opened this issue Jan 24, 2021 · 4 comments
Closed

Hide or shorten error codes in problems list #233

bwegge opened this issue Jan 24, 2021 · 4 comments
Assignees
Labels
1-feature-request ✨ Issue type: Request for a desirable, nice-to-have feature 3-fixed Issue resolution: Issue has been fixed on the develop branch
Milestone

Comments

@bwegge
Copy link

bwegge commented Jan 24, 2021

With most "Diagnostic Severity" levels, LTeX lists the identified spelling issues in the "Problems" tab in VS Code, where each issue consists of an explanation text plus some identifier string consisting of "LTeX - " and some all-caps code.

As these appended identifier strings are often rather long and all-caps, they take up a significant amount of space. What is more, when the "Problems" tab is narrow (e.g. when it's put on the side, not the bottom of the screen, see screenshot below), VS Code puts precedence on the identifier strings instead of the explanation, so while the codes are kept readable, the explanation is not. For a human user, this is quite counterintuitive since I'm really interested in the human-readable explanation and not the string "LTeX - " + code.

Is it possible to add an option to either shorten or entirely hide these identifier strings, or give precedence to the explanation instead of the identifier strings?

Thanks and best
image

@bwegge bwegge added the 1-feature-request ✨ Issue type: Request for a desirable, nice-to-have feature label Jan 24, 2021
@valentjn valentjn self-assigned this Jan 24, 2021
@valentjn valentjn added this to the 8.3.1 milestone Jan 24, 2021
@valentjn
Copy link
Owner

I moved the rule ID from the source of the diagnostic (that's what the part with the LTeX - is called) to the end of the message of the diagnostic, so that the rule ID will be truncated instead of the actual message of the rule. I think this is the best solution for most users. An additional setting shouldn't be necessary for this.

Regarding the visual layout, this should be a feature request for VS Code itself (see microsoft/vscode). LTEX can't control which one of source and message is truncated.

@valentjn valentjn added the 3-fixed Issue resolution: Issue has been fixed on the develop branch label Jan 24, 2021
@bwegge
Copy link
Author

bwegge commented Jan 24, 2021

That was quick, thanks for the prompt response (and also the extension, which is great).
The proposed solution sounds good for me, will gladly test the new release.

@valentjn
Copy link
Owner

Fix released in 8.4.0.

@bwegge
Copy link
Author

bwegge commented Feb 3, 2021

Works fine for me as expected, thanks a lot!

me-johnomar added a commit to me-johnomar/ltex-ls that referenced this issue Jan 31, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
1-feature-request ✨ Issue type: Request for a desirable, nice-to-have feature 3-fixed Issue resolution: Issue has been fixed on the develop branch
Projects
None yet
Development

No branches or pull requests

2 participants