[drupal-devel] [feature] Add vim modeline to have better support
for vim editor and code style.
willmoy
drupal-devel at drupal.org
Sun Aug 21 15:58:14 UTC 2005
Issue status update for
http://drupal.org/node/29310
Post a follow up:
http://drupal.org/project/comments/add/29310
Project: Drupal
Version: cvs
Component: base system
Category: feature requests
Priority: minor
Assigned to: Anonymous
Reported by: Cvbge
Updated by: willmoy
Status: patch (code needs review)
if you're editing the handbook, you might point out to people they could
just put the relevant settings in their .vimrc or, if they only want
these settings when editing drupal files they can alias say vid to "vi
-u ~/.vimrc-drupal" and have two configurations.
But this is a local configuration issue, not a drupal one.
willmoy
Previous comments:
------------------------------------------------------------------------
Sun, 21 Aug 2005 15:10:54 +0000 : Cvbge
Attachment: http://drupal.org/files/issues/vim.diff (23.56 KB)
Vim can be instructed to set special settings. The line added by this
patch tells vim that the file type is php, tab and 'shiftwidth' size is
2 characters and to expand tabs with spaces.
------------------------------------------------------------------------
Sun, 21 Aug 2005 15:13:47 +0000 : Cvbge
(as to not forget) This line can be easili added with sed -i -e
's!^?>$!// vim: ft=php ts=2 sw=2 et\n?>!'<files>
------------------------------------------------------------------------
Sun, 21 Aug 2005 15:15:49 +0000 : Morbus Iff
I'd -1 this. I don't use vi to code Drupal, and I'm sure a lot of other
people are using something entirely different as well. Allowing this
patch would allow the horde to say "oooh, oOh, add this line for MY
favorite text editor too!" - when would it stop? Best not to move
Drupal into this political territory.
------------------------------------------------------------------------
Sun, 21 Aug 2005 15:44:19 +0000 : Cvbge
Well, how many such editors there are? I can name vi* and emacs only...
------------------------------------------------------------------------
Sun, 21 Aug 2005 15:51:03 +0000 : chx
I feel inclined to say "won't fix". There is little use to add this. Add
that sed line to handbook and be done.
More information about the drupal-devel
mailing list