On Fri, Apr 21, 2017 at 04:39:03PM +0800, OOO wrote:
Actually nginx.vim supports on and off before. And some option also been highlighted because there are directives with the same name, ex: ssl. So if we complete remove this. It will looks like the function is missing.
Yes, but this highlighting is not directive-specific. Nothing stops from highlighting some parameters in all directives, and this is what I did in the PoC to preserve compatibility with the previous versions of highlighting.
On the other hand, I really think that highlighting of "on / off" in directives which do not handle these parameters is a misfeature, and in many cases it misleads people to think that a given directive does support on / off parameters.