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

Add "Why React" #48

Merged
merged 4 commits into from
Jun 3, 2013
Merged

Add "Why React" #48

merged 4 commits into from
Jun 3, 2013

Conversation

petehunt
Copy link
Contributor

@petehunt petehunt commented Jun 3, 2013

We need a way to communicate what we're actually doing rather than just how we build apps (like we do on the site.docs). This does a pretty good job of it, I think.


- **JavaScript is a flexible, powerful programming language with the ability to build abstractions.** This is incredibly important in large applications.
- "Logic" and "markup" are intimately tied, and are both part of the "presentation" layer, so we're not breaking separation of concerns.
- Large projects usually don't use WYSIWYG editors for production code
Copy link
Contributor

Choose a reason for hiding this comment

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

Not entirely sure how this motivates React. Maybe just tack on a "so…" clause to make the implication explicit?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

👍 will update

@vjeux
Copy link
Contributor

vjeux commented Jun 3, 2013

Add subtitles otherwise people won't bother reading 20 paragraphs
Would be nice to add illustrations/examples ofthe rendering phases so it's not only text

Nit: the author is being written as is, you want to put Pete Hunt

@petehunt
Copy link
Contributor Author

petehunt commented Jun 3, 2013

Updated per your comments, what illustrations do you think we need?

ability to build abstractions.** This is incredibly important in
large applications.
- "Logic" and "markup" are intimately tied, and are both part of the
"presentation" layer, so we're not breaking separation of concerns.
Copy link
Contributor

Choose a reason for hiding this comment

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

so we're not breaking separation of concerns, but rather unifying the presentation layer.

@petehunt
Copy link
Contributor Author

petehunt commented Jun 3, 2013

@zpao liked it

petehunt added a commit that referenced this pull request Jun 3, 2013
@petehunt petehunt merged commit 065f8ab into facebook:master Jun 3, 2013
large applications.
- "Logic" and "markup" are intimately tied, and are both part of the
"presentation" layer, so we're not breaking separation of concerns.
- Large projects usually don't use WYSIWYG editors for production
Copy link
Contributor

Choose a reason for hiding this comment

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

I feel like most people won't make the connection between WYSIWYG editors and splitting markup from code. I think something like:

Breaking apart markup from code is largely a historical problem. Historically, WYSIWYG editors like Dreamweaver would statically rebuild your markup each time your template changed. As a result, people would split their JS and CSS from their markup so that they could update them without rebuilding. Given that large projects don't generally use WYSIWYG editors for production code anymore, we've chosen to reduce friction by integrating rendering logic with markup.

We feel that each component should be a self contained atomic unit. In many other languages/syntaxes, you don't have separate completely disconnected files for describing presentation and behavior (think of Objective-C ViewControllers for instance).

is a bit closer to what I'd like (meh, I still hate the wording though - def. feel free to edit).

Also, fwiw, Marshall's original comment is here

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I like it, but I wish we could condense that down to a single bullet since this isn't our main point. Feel free to send a PR.

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

Successfully merging this pull request may close these issues.

4 participants