Fix matchit functionality

Glench / Vim-Jinja2-Syntax

An up-to-date jinja2 syntax file.

https://github.com/mitsuhiko/jinja2

BSD 3-Clause "New" or "Revised" License

343 stars 64 forks source link

Fix matchit functionality #5

Open icook opened 11 years ago

icook commented 11 years ago

This is an old file plugin that I managed to dig up that makes matchit work for Jinaj2 templates. There's probably some un-needed stuff in there, but it hasn't broken anything for me.

aidos commented 9 years ago

This is extremely useful. Would be great if it could be merged into the main project. For the moment I'm going to switch to icook's repo to get this behaviour..

ypid commented 9 years ago

:+1:

But I had some problems with matching the following:

{% if (apt is defined and apt) and apt != True %}
Acquire::http::Proxy "http://{{ apt }}:{{ apt_acng_port | default('3142') }}/";
Acquire::https::Proxy "false";
{% endif %}

So I am using the fork from chase who found a fix for this.

ernstki commented 2 years ago

@icook Was this PR meant to enable % matching on Jinja control flow like {% if … %} and {% for … %}, or just to get % working for HTML tags again?

If the latter, I think #19 already addressed % for HTML tags (works for me with the latest master anyway), in which case this PR could be closed.

icook commented 2 years ago

@ernstki Hard to remember honestly, but by my read it was specifically for Jinja2. The aforementioned fork from chase sounds like a more robust solution.