Only set 'syn-sync' when we're the main syntax

jparise / vim-graphql

A Vim plugin that provides GraphQL file detection, syntax highlighting, and indentation.

MIT License

487 stars 25 forks source link

Only set 'syn-sync' when we're the main syntax #72

Closed jparise closed 3 years ago

jparise commented 3 years ago

This should avoid problems when the GraphQL syntax is embedded within an outer syntax. In those cases, we don't want to change the syntax sync setting.

pjio commented 3 years ago

I tested the commit but it didn't solve the problem. I looked at other syntax plugins and it seems usual to set syn sync in the syntax script, so I made another PR which skips this part if the syntax is included as a nested syntax: https://github.com/jparise/vim-graphql/pull/73

jparise commented 3 years ago

Closing in favor of #73