Skip to content

Commit

Permalink
docs(Modal|Confirm): sort size prop in sizes order (#3237)
Browse files Browse the repository at this point in the history
docs(Modal|Confirm): sort `size` prop in sizes order
  • Loading branch information
fcwheat authored and layershifter committed Oct 29, 2018
1 parent 6e83af2 commit 236bcf8
Show file tree
Hide file tree
Showing 6 changed files with 6 additions and 6 deletions.
2 changes: 1 addition & 1 deletion src/addons/Confirm/Confirm.d.ts
Original file line number Diff line number Diff line change
Expand Up @@ -43,7 +43,7 @@ export interface StrictConfirmProps extends StrictModalProps {
open?: boolean

/** A confirm can vary in size. */
size?: 'fullscreen' | 'large' | 'mini' | 'small' | 'tiny'
size?: 'mini' | 'tiny' | 'small' | 'large' | 'fullscreen'
}

declare const Confirm: React.ComponentClass<ConfirmProps>
Expand Down
2 changes: 1 addition & 1 deletion src/addons/Confirm/Confirm.js
Original file line number Diff line number Diff line change
Expand Up @@ -44,7 +44,7 @@ class Confirm extends Component {
open: PropTypes.bool,

/** A Confirm can vary in size */
size: PropTypes.oneOf(['fullscreen', 'large', 'mini', 'small', 'tiny']),
size: PropTypes.oneOf(['mini', 'tiny', 'small', 'large', 'fullscreen']),
}

static defaultProps = {
Expand Down
2 changes: 1 addition & 1 deletion src/modules/Modal/Modal.d.ts
Original file line number Diff line number Diff line change
Expand Up @@ -101,7 +101,7 @@ export interface StrictModalProps extends StrictPortalProps {
open?: boolean

/** A modal can vary in size. */
size?: 'fullscreen' | 'large' | 'mini' | 'small' | 'tiny'
size?: 'mini' | 'tiny' | 'small' | 'large' | 'fullscreen'

/** Custom styles. */
style?: React.CSSProperties
Expand Down
2 changes: 1 addition & 1 deletion src/modules/Modal/Modal.js
Original file line number Diff line number Diff line change
Expand Up @@ -122,7 +122,7 @@ class Modal extends Component {
open: PropTypes.bool,

/** A modal can vary in size */
size: PropTypes.oneOf(['fullscreen', 'large', 'mini', 'small', 'tiny']),
size: PropTypes.oneOf(['mini', 'tiny', 'small', 'large', 'fullscreen']),

/** Custom styles. */
style: PropTypes.object,
Expand Down
2 changes: 1 addition & 1 deletion test/specs/addons/Confirm/Confirm-test.js
Original file line number Diff line number Diff line change
Expand Up @@ -54,7 +54,7 @@ describe('Confirm', () => {
shallow(<Confirm />).should.have.prop('size', 'small')
})

_.forEach(['fullscreen', 'large', 'mini', 'small', 'tiny'], (size) => {
_.forEach(['mini', 'tiny', 'small', 'large', 'fullscreen'], (size) => {
it(`applies ${size} size`, () => {
shallow(<Confirm size={size} />).should.have.prop('size', size)
})
Expand Down
2 changes: 1 addition & 1 deletion test/specs/modules/Modal/Modal-test.js
Original file line number Diff line number Diff line change
Expand Up @@ -217,7 +217,7 @@ describe('Modal', () => {
})

describe('size', () => {
const sizes = ['fullscreen', 'large', 'mini', 'small', 'tiny']
const sizes = ['mini', 'tiny', 'small', 'large', 'fullscreen']

sizes.forEach((size) => {
it(`adds the "${size}" to the modal className`, () => {
Expand Down

0 comments on commit 236bcf8

Please sign in to comment.