Commit cd752ab

ankurpjavivelasco

authored and

committed

Move removal of unused props into its own method to make it easy to subclass (#56)

1 parent 4befee0 commit cd752abCopy full SHA for cd752ab

File tree

+10

-3

lines changed

+10

-3

lines changed

Lines changed: 10 additions & 3 deletions

Original file line number	Diff line number	Diff line change
`@@ -105,6 +105,14 @@ export default (componentName, localTheme, options = {}) => (ThemedComponent) =>`
`105`	`105`	`: {}`
`106`	`106`	`}`
`107`	`107`
	`108`	`+ getPropsForComponent() {`
	`109`	`+ //exclude themr-only props`
	`110`	`+ //noinspection JSUnusedLocalSymbols`
	`111`	`+ const { composeTheme, innerRef, themeNamespace, ...props } = this.props //eslint-disable-line no-unused-vars`
	`112`	`+`
	`113`	`+ return props`
	`114`	`+ }`
	`115`	`+`
`108`	`116`	`getTheme(props) {`
`109`	`117`	`return props.composeTheme === COMPOSE_SOFTLY`
`110`	`118`	`? {`
`@@ -136,9 +144,8 @@ export default (componentName, localTheme, options = {}) => (ThemedComponent) =>`
`136`	`144`	`}`
`137`	`145`
`138`	`146`	`render() {`
`139`		`- //exclude themr-only props`
`140`		`- //noinspection JSUnusedLocalSymbols`
`141`		`- const { composeTheme, innerRef, themeNamespace, ...props } = this.props //eslint-disable-line no-unused-vars`
	`147`	`+ const { innerRef } = this.props`
	`148`	`+ const props = this.getPropsForComponent()`
`142`	`149`
`143`	`150`	`return React.createElement(ThemedComponent, {`
`144`	`151`	`...props,`

Comments

(0)