Skip to content

Fragment shorthand support #188

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

amadeus
Copy link

@amadeus amadeus commented Feb 19, 2019

This should fix #152

I left a comment in the source detailing my fix and why.

@amadeus
Copy link
Author

amadeus commented Feb 21, 2019

Any status on this? Wondering if it's worth just forking this repo if it's now considered inactive.

@0zguner
Copy link

0zguner commented Feb 22, 2019

@amadeus I'would like to use your fork. Can you also please merge other PRs if they are required ?

@amadeus
Copy link
Author

amadeus commented Feb 25, 2019

I have created a fork, merged this PR in, and a couple others. I am not 100% confident in a couple of the other PRs I merged in, but they looked ok. Feel free to give it a whirl.

amadeus/vim-jsx

I want to sit on it for a couple days before I start promoting it on vim-javascript.

@JESii
Copy link

JESii commented Apr 9, 2019

@amadeus -- I think it's time for you to start promoting your fork on vim-javascript. The indentation with AirBNB rules is a nightmare with this plugin. I started to fork it myself, then saw that you had done the same. I'm gonna switch to yours and see how it goes. Thanks!

@t-mdo
Copy link

t-mdo commented Apr 9, 2019

I went to vim-jsx-pretty.
It's pretty solid.

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Support Fragment Syntax
4 participants