search

zulus / extjs-eclipse

Eclipse plugin for extjs developing
11 stars 4 forks source link

trailing comma #4

Closed smasala1 closed 11 years ago

smasala1 commented 11 years ago

Hi there,

After a little fighting I got your plugin working and I have to say, it's so far faster than the sencha one! Excellent work.

What I have noticed though is that there is no trailing comma support as there is in the official plugin. (screenshot). Is there a possibility to enable this feature or is it no supported?

As you can see - there is no error message indicating that a trailing comma is present - when developing large web applications, it is almost impossible to find and IE has problems ignoring trailing commas.

screen

Thanks

zulus commented 11 years ago

Its still alpha version and haven't wizards yet ;)

For trailing comma and other problems, you should check http://github.eclipsesource.com/jshint-eclipse/

smasala1 commented 11 years ago

thanks - yes I will install JSHint for the moment, shame though as I found it quite heavy weight.

smasala1 commented 11 years ago

I got the trailing comma showing upo using jshint and es3: true - however in ExtJS it is considered an error rather than a warning.

zulus commented 11 years ago

I planned this, but while Sencha CMD integration.

Currently I'm working only on inferencing, code completion (configuration properties, listeners etc...) and wizards.

smasala1 commented 11 years ago

excellent! sounds good! its important for our Internet Explorer friends ;)

smasala1 commented 10 years ago

does this work now? (I've just seen your commit)

zulus commented 10 years ago

Its avaiable on snapshots repo: http://extjs.w3des.net/p2-dev/