fix(): Fixing the unused props issue with TimelineEvent

rcdexta · Nov 8, 2016 · 4e9050e · 4e9050e
1 parent e5cf1a2
commit 4e9050e
Showing 1 changed file with 4 additions and 6 deletions.
diff --git a/components/TimelineEvent.js b/components/TimelineEvent.js
@@ -3,17 +3,15 @@ import s from './styles'
 
 class TimelineEvent extends Component {
 
-  eventNotificationStyle () {
-    const {iconColor} = this.props
+  mergeNotificationStyle (iconColor) {
     if (iconColor) {
       return {...s.eventType, ...{color: iconColor, borderColor: iconColor}}
     } else {
       return s.eventType
     }
   }
 
-  contentStyle () {
-    const {contentStyle} = this.props
+  mergeContentStyle (contentStyle) {
     if (contentStyle) {
       return {...s.message, ...contentStyle}
     } else {
@@ -24,7 +22,7 @@ class TimelineEvent extends Component {
   render () {
     const {createdAt, title, contentStyle, buttons, icon, iconColor, ...otherProps} = this.props
     return <div style={s.event}>
-      <div style={this.eventNotificationStyle()}>
+      <div style={this.mergeNotificationStyle(iconColor)}>
         <span style={s.materialIcons}>{icon}</span>
       </div>
       <div style={s.eventContent} {...otherProps}>
@@ -33,7 +31,7 @@ class TimelineEvent extends Component {
         <div style={s.title}>{title}</div>
         <div style={s.actionButtons}>{buttons}</div>
         <div style={{clear: 'both', marginBottom: 10}} />
-        <div style={this.contentStyle()}>
+        <div style={this.mergeContentStyle(contentStyle)}>
           {this.props.children}
           <div style={s.messageAfter} />
         </div>