search

vim-pandoc / vim-pandoc-syntax

pandoc markdown syntax, to be installed alongside vim-pandoc
MIT License
424 stars 61 forks source link

smart squotes and apostrophe #313

Closed rwxrob closed 4 years ago

rwxrob commented 4 years ago

Hey there, would you mind giving me an update as to the possibility of this being accepted? If it is not something you are interested in I understand. I just want to know if I should reconfigure my lab systems to use my fork or remain with your main one. Thank you.

alerque commented 4 years ago

Sorry this is just a big change to review and I haven't been on the right systems to do it. Also next time please make changes in the original branch, opening new PR's makes it hard to follow the history and see what changed between reviews.

That being said I think this looks good overall. I'm sure we'll be able to include it here so no need to fork. There might be a few things to tweak first. Let me try to get it running locally.

rwxrob commented 4 years ago

This is not an enhancement. As you will find many of my changes correct omissions in ranges that are expected.

I am closing this ticket and will be maintaining my own fork from now on and adding several other critical formatting and color fixes to those you have already rejected, as well as several other things that have bothered me for some time:

These changes and others constitute a significant diversion from the existing project so I feel forking is best. In fact, because I will be removing a lot of functionality and catering to my simplified (Ezmark) Pandoc variation I'll change the name to vim-pandoc-syntax-ez instead. It is 100% compatible with Pandoc Markdown but removes redundancy reducing the time to learn. (I work with many young people learning Pandoc and Markdown for the first time.)

Hopefully someone will find my fixes helpful and decide one day to implement them in whatever way seems best to them. Thanks for your consideration and time.

alerque commented 4 years ago

You are welcome to maintain your own fork with tweaks suitable for your usage. But at the same time we do still want to pull in the fixes and improvements you've presented here. The only request we made is that it not also mess with overriding themes. If you don't want to make that change to contribute here I'm happy to fix it up myself, and then you can fork with only a minimal set of changes including the theming bit.

alerque commented 4 years ago

(PR re-closed because I just realized it is against your master branch and I can't pull in the latest or fix stuff there, so I'll had to do this in another branch).