tamzinblake / js3-mode

A chimeric fork of js2-mode and js-mode
GNU General Public License v3.0
180 stars 13 forks source link

Operators not lining up properly #5

Closed tamzinblake closed 13 years ago

tamzinblake commented 13 years ago

It does:

    if (success_ajax(response, options)) {
      var imgdom = Ext.get('previewImage').dom;
      imgdom.src = imgdom.src.replace(/\?.+/, '')
                    + '?' + (new Date()).getTime();
    }

Should be:

    if (success_ajax(response, options)) {
      var imgdom = Ext.get('previewImage').dom;
      imgdom.src = imgdom.src.replace(/\?.+/, '')
                 + '?' + (new Date()).getTime();
    }

Note it is lining up under the paren for success_ajax(

tamzinblake commented 13 years ago

The following is also problematic:

It does:

if (success_ajax(response, options)) {
  var imgdom = Ext.get('previewImage').dom;
  imgdom.src = (imgdom.src.replace(/\?.+/, '')
  , '?' + (new Date()).getTime());
}

Should be:

if (success_ajax(response, options)) {
  var imgdom = Ext.get('previewImage').dom;
  imgdom.src = (imgdom.src.replace(/\?.+/, '')
               , '?' + (new Date()).getTime());
}
tamzinblake commented 13 years ago

fixed by commit 8ece232