Commit Graph
1113 Commits
Author SHA1 Message Date
Ben Alpert 0e2840abce Update commoner to 0.8.8 for Windows support
benjamn/commoner#44 fixed commoner to work on Windows when module path relativization isn't used; with this, the `jsx` binary should work properly on Windows (though `jsx-internal` still won't).

Fixes #316, fixes #391, fixes #567.
2013-11-24 16:02:01 -05:00
Pete Hunt d51ae6b8bc Merge pull request #575 from spicyj/owner
Move __owner__ off of props
2013-11-21 17:44:48 -08:00
Ben Alpert 61abc645e5 Move __owner__ off of props 2013-11-21 17:34:52 -08:00
Ben Newman 2a66c9b089 Merge pull request #586 from benjamn/reinstate-ReactEventEmitter-mocking
Add back mocking removed to make ReactEventTopLevelCallback-test pass.
2013-11-21 11:07:57 -08:00
Ben Newman 52d3b47f48 Add back mocking removed to make ReactEventTopLevelCallback-test pass.
This is a partial reversion of commit 9937f0c8bd.
2013-11-21 10:57:24 -08:00
Ben Newman 97bbd852b2 Merge pull request #584 from spicyj/mock
Mock modules properly in test runner.
2013-11-21 10:31:59 -08:00
Ben Alpert af95b35f27 Have faith in var hoisting 2013-11-21 10:31:20 -08:00
Ben Alpert c7bb3af760 Mock modules properly in test runner
As an added bonus, the jasmine web interface now groups tests by file.

Test Plan:
grunt test passes on b2507066, the parent of 566f8b2e (which committed a workaround for buggy module mocking).
2013-11-21 00:39:37 -08:00
Ben Newman 566f8b2e85 Merge pull request #573 from spicyj/tests
Make ReactEventTopLevelCallback-test pass
2013-11-20 16:54:15 -08:00
Paul O’Shannessy b2507066b6 Don't call utils.traverse in transform
Accidental change we missed in review of #495.
2013-11-20 15:36:54 -08:00
Christopher Chedeau e73900dad4 Remove rootNode from componentDidMount and componentDidUpdate
We can already access the DOM node using `this.getDOMNode()`. Passing it as an argument was a bad decision.

Previous API:

```
componentDidMount(DOMElement rootNode)
componentDidUpdate(object prevProps, object prevState, object prevContext, DOMElement rootNode)
```

Next API:

```
componentDidMount()
componentDidUpdate(object prevProps, object prevState, object prevContext)
```
2013-11-20 14:20:35 -08:00
Christoph Pojer 2fe2cd5337 Ensure ReactPerf always uses a string as a URL 2013-11-20 14:19:13 -08:00
Tim Yung 04c3e2e407 Add nextContext to componentWillReceiveProps
Add `nextContext` as an argument to `componentWillReceiveProps`. We can figure out what to rename the method to later.
2013-11-20 14:19:01 -08:00
Christopher Chedeau a23d43bf05 Use this.getDOMNode() instead of last argument of componentDidMount 2013-11-20 14:18:49 -08:00
Marshall Roch 934ef1d4c2 Fix context in callbacks
callbacks like shouldComponentUpdate, componentDidUpdate, etc. were getting the full, unsanitized context, instead of the one that's been filtered down to only the fields allowed to be visible by contextTypes.
2013-11-20 14:18:28 -08:00
Christopher Chedeau c8b6fe51d9 Use this.getDOMNode() instead of last argument of componentDidUpdate
Context is adding another argument that shifts all of them by one. Since we can already use this.getDOMNode(), it doesn't really make sense to pass it as an argument to that function.
2013-11-20 14:18:07 -08:00
Marshall Roch 3fd3341ab9 remove transaction argument from componentWillReceiveProps 2013-11-20 14:17:23 -08:00
Paul O'Shannessy 060118c7e4 order object properties consistently 2013-11-20 14:17:13 -08:00
Pete Hunt 3651b8892f Merge pull request #578 from spicyj/patch-2
autoBind -> Autobinding
2013-11-20 11:47:22 -08:00
Ben Alpert d49d84b250 autoBind -> Autobinding
We don't use the term autoBind anywhere any more.
2013-11-20 11:42:05 -08:00
Pete Hunt 0df4be849f Merge pull request #577 from stillmotion/master
Add explination of autoBind to DOM Event Listener tip
2013-11-20 11:41:11 -08:00
Levi McCallum ceaf3fba32 Add explination of autoBind to DOM Event Listener tip 2013-11-20 11:29:05 -08:00
Ben Alpert 9937f0c8bd Make ReactEventTopLevelCallback-test pass
This module-level mock() seems to have been interfering with other tests (26 specs failing). We don't have any other tests that do a module-level mock() so I'm going to assume it isn't supported in the open-source test runner right now. I changed it so only ReactEventEmitter.handleTopLevel is mocked; doing so made ReactEventEmitter complain that TopLevelCallbackCreator wasn't defined so I switched the ReactMount references to use React directly so that ReactDefaultInjection would kick in properly.

With this, all the tests pass.
2013-11-19 23:46:32 -08:00
Paul O’Shannessy bd1f5e7e16 Put nav data in "_data"
New in Jekyll 1.3 - http://jekyllrb.com/docs/datafiles/
2013-11-19 15:52:42 -08:00
Paul O’Shannessy f37fd7a7a3 Fix pagination 2013-11-19 15:25:00 -08:00
Paul O’Shannessy 378a49bf3c Update jekyll to 1.3 2013-11-19 15:18:15 -08:00
Vjeux 96e97c1a87 Community round-up #11 2013-11-19 22:56:34 +01:00
Paul O’Shannessy 5f22259964 Merge pull request #565 from chenglou/className-htmlFor
docs highlight className and htmlFor transforms
2013-11-19 11:01:10 -08:00
Jeff Morrison ab47e99215 Merge pull request #495 from syranide/fixutils
utils.* is now used everywhere
2013-11-19 10:59:16 -08:00
Cheng Lou 1fcd6412fb docs highlight className and htmlFor transforms 2013-11-18 21:42:51 -05:00
Cheng Lou ce0eec97db docs classSet semicolons missing 2013-11-18 21:22:14 -05:00
Paul O’Shannessy 5d2f0b4e07 Merge branch 'chenglou-classSet'
closes #463
2013-11-18 18:00:24 -08:00
Paul O’Shannessy e79079d174 Merge pull request #539 from spicyj/doc-anchors-again
Make doc headers clickable again
2013-11-18 17:45:39 -08:00
Ben Alpert e94ebee15e Make doc headers clickable again
...without preventing clicks on other things.

Just use an `<a name="...">` tag that doesn't take up any space to make sure that we're not covering up something else.

For whatever reason, doing `position: relative; top: -$navHeight;` doesn't work and causes the anchor target not to be moved up. This solution works in both Chrome and Firefox.
2013-11-18 17:34:44 -08:00
Ben Newman 5d878ce914 Merge pull request #550 from subtleGradient/subtlegradient/code-coverage
Code coverage grunt tasks.
2013-11-18 14:03:56 -08:00
Thomas Aylott 5ae152cdcf simplified the sort function 2013-11-18 16:37:47 -05:00
Thomas Aylott c6f7fe00fa Lines too long; reformatted 2013-11-18 16:37:47 -05:00
Thomas Aylott 45063aed44 picked all the lint 2013-11-18 16:37:47 -05:00
Thomas Aylott 5feb745b02 fixes the case when coverage isn't turned on 2013-11-18 16:37:47 -05:00
Thomas Aylott eda56b7af2 new grunt test:coverage task 2013-11-18 16:37:47 -05:00
Thomas Aylott 8f96ec255b new grunt browserify:withCodeCoverageLogging task 2013-11-18 16:37:47 -05:00
Thomas Aylott ad0d9e4761 consoleLoggerMiddleware handles coverage logs 2013-11-18 16:37:47 -05:00
Thomas Aylott b5b60a6acf speed up server middleware slightly 2013-11-18 16:37:46 -05:00
Thomas Aylott cb6b7f37e7 fixes an issue where the list of files to test isn't complete 2013-11-18 16:37:46 -05:00
Thomas Aylott ef5a02c164 enable coverage logging from the worker 2013-11-18 16:37:46 -05:00
Thomas Aylott 646421f71f batch logs until the end unless ran with --debug 2013-11-18 16:37:46 -05:00
Thomas Aylott 4c881d8487 require coverify 2013-11-18 16:37:46 -05:00
Thomas Aylott 5aa901336c ignore logs and testing stuff 2013-11-18 16:37:46 -05:00
Thomas Aylott 61c1bf0a41 fixes browserify task transforms support 2013-11-18 16:37:46 -05:00
Daniel Lo Nigro d853c8568e Propagate events to parent components when nested
When events are captured, the nearest React-rendered ancestor is found and events are propagated to its tree. This causes issues when components are nested as only the innermost component receives the events.

This change modifies this behaviour so events propagate all the way up the DOM hierarchy. To reduce the performance cost, this DOM traversal is only done if the component is actually nested (determined by the `containerIsNested` map which is lazily initialised).
2013-11-18 10:56:29 -08:00