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

Prevent CodeLens CSS selector name from being invalid #95005

Merged
merged 2 commits into from
Apr 14, 2020

Conversation

x24git
Copy link
Contributor

@x24git x24git commented Apr 11, 2020

This PR fixes #94958

In the codelensController, there is an issue when the class name for the codeLens span element gets generated on line 70.

this._styleClassName = hash(this._editor.getId()).toString(16); 

Adding an underscore in front of the hashed editor ID should allow any id value to accepted as a valid CSS selector (since underscores are always allowed to start CSS selectors).

@jrieken jrieken merged commit 4f1ad60 into microsoft:master Apr 14, 2020
@jrieken jrieken added this to the April 2020 milestone Apr 14, 2020
@github-actions github-actions bot locked and limited conversation to collaborators May 29, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

CodeLens allows for invalid classnames
2 participants