Skip to content

Add sparkup for Zen Coding. #396

Closed
wants to merge 1 commit into from

3 participants

@Gaelan
Gaelan commented Jul 5, 2013

Sparkup's default mappings conflict with our NERDTree mappings. Will fix in later commit. Fixes #392.

@Gaelan Gaelan Add sparkup for Zen Coding.
Sparkup's default mappings conflict with our NERDTree mappings. Will fix
in later commit.
7b79099
@spf13
Owner
spf13 commented Oct 12, 2013

Happy to merge, but waiting on the later commit first... Is it coming?

@Gaelan
Gaelan commented Oct 12, 2013

@spf13 I just don't know what mapping I should use.

@johntyree
Collaborator

@Gaelan This PR is dead in the water without a solution for the mapping conflict. Preferably one that doesn't surprise people that don't know or care about sparkup.

What does this actually do? What are the bindings for? Can they be ignored entirely?

@Gaelan
Gaelan commented Nov 30, 2013

It's a binding to expand the zencoding syntax to full HTML.

@johntyree johntyree commented on the diff Dec 1, 2013
.vimrc.bundles
@@ -196,6 +196,9 @@
Bundle 'amirh/HTML-AutoCloseTag'
Bundle 'hail2u/vim-css3-syntax'
Bundle 'tpope/vim-haml'
+ if (has("python") || has("python3")) && !exists('g:spf13_use_old_powerline')
@johntyree
Collaborator
johntyree added a note Dec 1, 2013

Does this depend on powerline somehow? What about people using airline?

@Gaelan
Gaelan added a note Dec 16, 2013

Sparkup uses python for multi-editor support, sort of like powerline does, so I copied the condition for powerline. I should probably get rid of the last condition.

@johntyree
Collaborator
johntyree added a note Dec 23, 2013

Please do.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
@spf13 spf13 closed this Jun 4, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Something went wrong with that request. Please try again.