Skip to content
This repository was archived by the owner on Sep 14, 2023. It is now read-only.

Commit 0c2ed8f

Browse files
committed
Removed BrowserModal indirection and merged BrowserModal w/ Modal
- fixes #35 - Added prop `className` to `Modal` (fixes #50) - Added prop `wrapperClassName` to `Modal` - Added prop `modalClassName` to `Modal.Dialog` - Added node 9 to supported development engines
1 parent 665841a commit 0c2ed8f

21 files changed

+179
-191
lines changed

CHANGELOG.md

+9
Original file line numberDiff line numberDiff line change
@@ -22,6 +22,15 @@
2222

2323
<!-- END doctoc generated TOC please keep comment here to allow auto update -->
2424

25+
### v0.9.0-dev
26+
27+
- Removed BrowserModal indirection and merged BrowserModal w/ Modal (fixes #35)
28+
- Added prop `className` to `Modal` (fixes #50)
29+
- Added prop `wrapperClassName` to `Modal`
30+
- Added prop `modalClassName` to `Modal.Dialog`
31+
- Added node 9 to supported development engines
32+
33+
2534
### v0.8.0
2635

2736
- :exclamation: Drop support for react `0.14`

README.md

+2
Original file line numberDiff line numberDiff line change
@@ -67,7 +67,9 @@ This will render a static modal, which cannot be hidden by the user.
6767
| onToggle | func | Function called after the modal is toggled. `function (isOpen, props) { }`
6868
| transitionName | string | Name of animation to use for open/close (to see how to define custom ones, see default styles) [default: tg-modal-fade]
6969
| transitionDuration | int | Duration of the transition in milliseconds [default: 300]
70+
| className | string | Extra classnames to use for modal [default: ``]
7071
| dialogClassName | string | Classname to use for `ModalDialog` [default: tg-modal-dialog]
72+
| wrapperClassName | string | Extra classnames to use for modal wrapper [default: ``]
7173

7274
Props not specified here are considered internal, and are prone to change.
7375

collect.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,7 @@ var fs = require('fs');
1111
*/
1212
function transformFile(contents) {
1313
// Show prettier import paths
14-
contents = contents.replace('../../../src/browser', 'tg-modal');
14+
contents = contents.replace('../../../src/components/Modal', 'tg-modal');
1515

1616
return contents;
1717
}

examples/components/Serverside.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
import PropTypes from 'prop-types';
22

3-
import Modal from '../../src/browser';
3+
import Modal from '../../src/components/Modal';
44

55

66
// Kiosk is like store+actions in the same class

examples/components/examples/Basic.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
import React, { Component } from 'react';
22
import PropTypes from 'prop-types';
33

4-
import Modal from '../../../src/browser';
4+
import Modal from '../../../src/components/Modal';
55

66

77
class BasicModalExample extends Component {

examples/components/examples/BasicConfirm.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
import React, { Component } from 'react';
22
import PropTypes from 'prop-types';
33

4-
import Modal from '../../../src/browser';
4+
import Modal from '../../../src/components/Modal';
55

66

77
class BasicConfirmModalExample extends Component {

examples/components/examples/Confirm.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
import React, { Component } from 'react';
22
import PropTypes from 'prop-types';
33

4-
import Modal from '../../../src/browser';
4+
import Modal from '../../../src/components/Modal';
55

66

77
class ConfirmModalExample extends Component {

examples/components/examples/Long.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
import React, { Component } from 'react';
22
import PropTypes from 'prop-types';
33

4-
import Modal from '../../../src/browser';
4+
import Modal from '../../../src/components/Modal';
55

66

77
class LongModalExample extends Component {

examples/components/examples/Markdown.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,7 @@ import ReactMarkdown from 'react-markdown';
55

66
import theMarkdown from '../../files/example.md';
77

8-
import Modal from '../../../src/browser';
8+
import Modal from '../../../src/components/Modal';
99

1010

1111
class MarkdownModalExample extends Component {

examples/components/examples/NestedLong.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,7 @@ import PropTypes from 'prop-types';
33

44
import classNames from 'classnames';
55

6-
import Modal from '../../../src/browser';
6+
import Modal from '../../../src/components/Modal';
77

88

99
class LongNestedModalExample extends Component {

examples/components/examples/ReduxModal.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,7 @@ import PropTypes from 'prop-types';
33
import { connect, Provider } from 'react-redux';
44
import { createStore } from 'redux';
55

6-
import Modal from '../../../src/browser';
6+
import Modal from '../../../src/components/Modal';
77

88

99
// redux actions and types

examples/components/examples/Simple.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
import React, { Component } from 'react';
22
import PropTypes from 'prop-types';
33

4-
import Modal from '../../../src/browser';
4+
import Modal from '../../../src/components/Modal';
55

66

77
class SimpleModalExample extends Component {

examples/components/examples/WithComponents.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
import React, { Component } from 'react';
22
import PropTypes from 'prop-types'
33

4-
import Modal from '../../../src/browser';
4+
import Modal from '../../../src/components/Modal';
55

66

77
class WithComponentsModalExample extends Component {

package-lock.json

+10-10
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

package.json

+1-1
Original file line numberDiff line numberDiff line change
@@ -113,7 +113,7 @@
113113
"check-engine": "node node_modules/fbjs-scripts/node/check-dev-engines.js package.json"
114114
},
115115
"devEngines": {
116-
"node": "4.x || 5.x || 6.x || 7.x || 8.x",
116+
"node": "4.x || 5.x || 6.x || 7.x || 8.x || 9.x",
117117
"npm": "2.x || 3.x || 4.x || 5.x"
118118
},
119119
"nyc": {

src/browser.js

-151
This file was deleted.

0 commit comments

Comments
 (0)