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

ambigous html code highlighting #150679

Closed
noontz opened this issue May 30, 2022 · 3 comments
Closed

ambigous html code highlighting #150679

noontz opened this issue May 30, 2022 · 3 comments
Assignees
Labels
info-needed Issue requires more information from poster

Comments

@noontz
Copy link

noontz commented May 30, 2022

Issue Type: Bug

1.Add a button in a html document
2.Add an onclick listener inline
3. repeat 1. 2.

VS Code version: Code 1.67.2 (c3511e6, 2022-05-17T18:15:52.058Z)
OS version: Windows_NT x64 10.0.19044
Restricted Mode: No

System Info
Item Value
CPUs Intel(R) Core(TM) i5-4440 CPU @ 3.10GHz (4 x 3100)
GPU Status 2d_canvas: enabled
canvas_oop_rasterization: disabled_off
direct_rendering_display_compositor: disabled_off_ok
gpu_compositing: enabled
multiple_raster_threads: enabled_on
oop_rasterization: unavailable_off
opengl: enabled_on
rasterization: unavailable_off
raw_draw: disabled_off_ok
skia_renderer: enabled_on
video_decode: enabled
video_encode: enabled
vulkan: disabled_off
webgl: enabled
webgl2: enabled
Load (avg) undefined
Memory (System) 15.93GB (8.05GB free)
Process Argv --crash-reporter-id 1e6d5089-6165-45ec-844d-5f2ea6a18690
Screen Reader no
VM 0%
Extensions (7)
Extension Author (truncated) Version
Bookmarks ale 13.3.0
vscode-eslint dba 2.2.2
svg joc 1.4.18
vetur oct 0.35.0
LiveServer rit 5.7.5
comment-ts sal 1.0.21
shader sle 1.1.5
A/B Experiments
vsliv368:30146709
vsreu685:30147344
python383:30185418
vspor879:30202332
vspor708:30202333
vspor363:30204092
pythonvspyl392:30443607
pythontb:30283811
pythonptprofiler:30281270
vshan820:30294714
vstes263:30335439
vscorecescf:30445987
pythondataviewer:30285071
vscod805:30301674
pythonvspyt200:30340761
binariesv615:30325510
bridge0708:30335490
bridge0723:30353136
vsaa593cf:30376535
vsc1dst:30438360
pythonvs932:30410667
wslgetstarted:30449410
pythonvsnew555:30457759
vscscmwlcmt:30465135
cppdebug:30492333
vscaac:30438847
vsclangdf:30486550

vs_code_bug

@aeschli
Copy link
Contributor

aeschli commented May 30, 2022

Can you paste the code snippet to reproduce?
I was not able to do so with the following (similar) snippet:

<div>
	<button onclick="goBack()">Back</button>
	<button onclick="openUsers()">Users</button>
	<span</span>
	<button onclick="logOut()">LogOut</button>
	<button onclick="logOut()">LogOut</button>
</div>

@aeschli aeschli added the info-needed Issue requires more information from poster label May 30, 2022
@noontz
Copy link
Author

noontz commented Jun 2, 2022 via email

@aeschli
Copy link
Contributor

aeschli commented Jun 2, 2022

Ah, now I see the problem, the second onClick attribute uses a upper case C.

This is a bug in the HTML grammar and the issue is: textmate/html.tmbundle#104

Closing as duplicate

@aeschli aeschli closed this as completed Jun 2, 2022
@github-actions github-actions bot locked and limited conversation to collaborators Jul 17, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
info-needed Issue requires more information from poster
Projects
None yet
Development

No branches or pull requests

2 participants