Brackets-InteractiveLinter icon indicating copy to clipboard operation
Brackets-InteractiveLinter copied to clipboard

</source> is not needed

Open PercyButton opened this issue 7 years ago • 5 comments

Linter prompted: Tag must be paired, Missing: [ ] with the following code:

        <video id="1" width="560" height="315" controls>
            <source src="new/How the Tesla Model S is Made _ Tesla Motors Part 1 (WIRED).mp4" type="video/mp4">            
            Please upgrade your browser!
        </video>

But the close tag is not necessary: https://developer.mozilla.org/en-US/docs/Web/HTML/Element/source

PercyButton avatar May 17 '17 01:05 PercyButton

Hi, so what was the answer here as i have exactly the same problem. Iv just started using brackets and learning html,css,js,php and would appreciated some advice. Thx.

JackDinn avatar Sep 12 '17 23:09 JackDinn

Oh yeah for sure. Always close tags, even if not required. Those cases are the exception and from my experience you would be doing it because you can and not because you should. Its kinda like using loose equal comparison vs strict comparison in JavaScript. Its allowed, but don't do it. :)

Hope that helps.

MiguelCastillo avatar Sep 13 '17 00:09 MiguelCastillo

did i do this wrong ? i tried closing the tags, the interactive linter is ok now (showing green) but the default syntax checker still shows RED , im assuming a lot here cos iv only just started out but heres a screen shot https://imgur.com/W7wuUOI

JackDinn avatar Sep 13 '17 00:09 JackDinn

does the default checker ever go away? what if you remove the video tag - just for testing purposes?

MiguelCastillo avatar Sep 13 '17 01:09 MiguelCastillo

well i was not 100% sure if it was actually the default linter/checker or not, it could have been one of my extensions since i added a load that looked like id be wanting them before i even used brackets. So i disabled all the addons. Looks like it is the default error checker and yes it does go away when i remove the closing tag. Must be a setting to turn the default syntax checking off somewhere? maybe :-/ 4 SS's here https://imgur.com/a/U8xHi

JackDinn avatar Sep 13 '17 02:09 JackDinn