cleanup and few changes

mrmartineau / HTML5

HTML5 bundle for Sublime Text

193 stars 52 forks source link

cleanup and few changes #16

Closed ctf0 closed 9 years ago

ctf0 commented 9 years ago

changed the tmlang name from HTML5 to HTML for 3 reasons

because laravel blade cant pickup the right HL when using HTML5 so the default HTML have to be enabled
the need to keep changing between HTML5 & HTML ,now u dont
no more need to disable the default HTML package as now this will override the default and Sublime linter can now work with the new syntax instead of the default.

edited the form input names for easier & faster completion. removed any snippet < IE8

mrmartineau commented 9 years ago

@ctf0, thanks for your input. I have not seen all of the changes you have made so I cannot be 100% certain on this, but I won't be merging the tabTrigger changes you have made because I believe it is easier to type link and see all the options for link rather than remembering the different shortcuts. Do you see?

Does removing some of the HTML5 tags break it or is have you done that because Sublime's default HTML package handles these tags?

I will try to comment on some of the other changes inline when I have more time.

ctf0 commented 9 years ago

for the links it still works as b4 ,u just got a one more character in case u dont want to scroll the whole menu to find what u want ,same goes with the input.

i didn't remove any of the html tags ,i just changed the language file from being HTML5 to HTML because laravel blade works only with HTML and cant recognize the HTML5 + the other reasons i mentioned above.

mrmartineau commented 9 years ago

Hi @ctf0, after examining this more thoroughly I will merge your changes. Your reasons for the tabtrigger changes make a lot of sense. Thanks for your help.

ctf0 commented 9 years ago

np ,glad to help :).