madskristensen / RainbowBraces

A Visual Studio extension
Apache License 2.0
134 stars 7 forks source link

Braces and brackets are not coloured in vbhtml or html files #72

Closed graymatter00 closed 1 year ago

graymatter00 commented 1 year ago

Braces, brackets and vertical lines are not coloured in vbhtml or regular html files. See screenshots below. All other files seem to be coloured as expected.

vbhtml file...

image

image

image

image

image

graymatter00 commented 1 year ago

Unfortunately, it isn't working quite right.

image

Is this comment enough to re-open the issue or would you like me to create a new issue?

Also, regular HTML files (.htm) are not being colored at all.

image

lordfanger commented 1 year ago

The @Url.Action("Index", "Contact") could be fixed in next release after merge #83 I'm not sure about the rest. It works ok i my test files. Can you attach whole files that don't work? Or are you using some other extension that can interfere? (custom HTML editor, another colorizer)

First screen looks like after first occurence of inline code it stops working.

graymatter00 commented 1 year ago

@lordfanger Yes, the new version containing merger #83 fixed the .vbhtml issue. It's now coloured as I'd expect. Thanks! 😊

The .htm files are still not being coloured. I'll do some more checking and raise it as a separate issue in the next day or so.

🖖

lordfanger commented 1 year ago

Sure, and try this. Right click on .htm file and select option Open With... and post what editor is set as (Default). There might be some editor that is not explicitly allowed by this extension. If it is so, it should be enabled by only a simple edit.

image image

graymatter00 commented 1 year ago

@lordfanger Sorry for the delay in replying. Life got in the way.

As requested, attached are two screenshots showing the Open With dialog window for both an existing .htm file and a newly created .html file.

image

image

lordfanger commented 1 year ago

I don't see something suspicious. But added support for Web Forms in #92 PR. Maybe it could help if Mads merges it.