Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

Update self-closing-comp doc to show example of a intentional space #745

Merged
merged 1 commit into from Aug 7, 2016

Conversation

petersendidit
Copy link
Contributor

This came up when reviewing #730

@ljharb
Copy link
Member

ljharb commented Aug 6, 2016

I think the added example is good 👍 but I don't think it resolves #730 whatsoever.

@@ -15,9 +15,13 @@ The following patterns are not considered warnings:
```js
var contentContainer = <div className="content"></div>;

var intentionalSpace = <div>{' '}</div>;
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It would be nice to include this example in the html section down below.

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Also the component section, I guess.

@lencioni lencioni merged commit c0038f9 into jsx-eslint:master Aug 7, 2016
@petersendidit petersendidit deleted the self-closing-comp-docs branch August 8, 2016 14:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants