Improved Javascript indenting and syntax for inclusion

273 views
Skip to first unread message

Darrick Wiebe

unread,
Apr 3, 2014, 1:36:30 PM4/3/14
to vim...@googlegroups.com
Hi all,

The vim-javascript plugin at https://github.com/pangloss/vim-javascript is significantly better than the one built in to Vim, which I am told is currently unmaintained. Vim-javascript has an active group of contributors and maintainers, while the indenter works well and has been mostly stable since 2009.

Tim Pope recently created an issue on the project suggesting that we contribute it to Vim for inclusion in the next release. I think that's a good idea and probably should have been done long ago.

I hope you'll agree. Regards,

Darrick Wiebe (pangloss)

David Fishburn

unread,
Jun 11, 2014, 11:49:44 AM6/11/14
to vim_dev
Darrick,

I have been using this version (daily) since you sent out this email and found it to be a significant improvement.  So thank you.

Couple of comments:

1.  You have a plugin for this already here:
Better Javascript Indentation : Vastly improved javascript indentation http://www.vim.org/scripts/script.php?script_id=2765

Could you update that version with this one.

2.  I knew I had recently downloaded a new Javascript indent script and was trying to figure out where I got it from.  This is the top of the indent/javascript.vim file:
" Vim indent file
" Language: Javascript
" Acknowledgement: Based off of vim-ruby maintained by Nikolai Weibull http://vim-ruby.rubyforge.org

This doesn't tell me much, and it took me some digging to find this email and the web site, plus diffing a bunch of different javascript.vim files to determine this is one I am using.


This ties into #1.  If you could add headers, plus authors, plus how to contact the author (for suggestions and bug) to the top of each file included in this update it would be quite useful.

I believe all this information will be required before Bram would include it in the official release as well.

Thanks for new scripts, great improvement.

David


Reply all
Reply to author
Forward
0 new messages