small changes

mui · Nov 7, 2018 · 9a59daf · 9a59daf
1 parent 96ea97f
commit 9a59daf
Show file tree

Hide file tree

Showing 6 changed files with 49 additions and 55 deletions.
diff --git a/packages/material-ui/src/Table/Table.js b/packages/material-ui/src/Table/Table.js
@@ -17,10 +17,9 @@ export const styles = theme => ({
 
 function Table(props) {
   const { classes, className, component: Component, padding, ...other } = props;
-  const childContext = { padding };
 
   return (
-    <TableContext.Provider value={childContext}>
+    <TableContext.Provider value={{ padding }}>
       <Component className={classNames(classes.root, className)} {...other} />
     </TableContext.Provider>
   );

diff --git a/packages/material-ui/src/Table/Tablelvl2Context.js b/packages/material-ui/src/Table/Tablelvl2Context.js
@@ -3,6 +3,6 @@ import React from 'react';
 /**
  * @ignore - internal component.
  */
-const TableContext = React.createContext();
+const Tablelvl2Context = React.createContext();
 
-export default TableContext;
+export default Tablelvl2Context;
diff --git a/packages/material-ui/src/TableBody/TableBody.js b/packages/material-ui/src/TableBody/TableBody.js
@@ -13,10 +13,9 @@ export const styles = {
 
 function TableBody(props) {
   const { classes, className, component: Component, ...other } = props;
-  const childContext = { variant: 'body' };
 
   return (
-    <Tablelvl2Context.Provider value={childContext}>
+    <Tablelvl2Context.Provider value={{ variant: 'body' }}>
       <Component className={classNames(classes.root, className)} {...other} />
     </Tablelvl2Context.Provider>
   );

diff --git a/packages/material-ui/src/TableCell/TableCell.js b/packages/material-ui/src/TableCell/TableCell.js
@@ -85,55 +85,53 @@ function TableCell(props) {
 
   return (
     <TableContext.Consumer>
-      {table => {
-        return (
-          <Tablelvl2Context.Consumer>
-            {tablelvl2 => {
-              let Component;
-              if (component) {
-                Component = component;
-              } else {
-                Component = tablelvl2 && tablelvl2.variant === 'head' ? 'th' : 'td';
-              }
+      {table => (
+        <Tablelvl2Context.Consumer>
+          {tablelvl2 => {
+            let Component;
+            if (component) {
+              Component = component;
+            } else {
+              Component = tablelvl2 && tablelvl2.variant === 'head' ? 'th' : 'td';
+            }
 
-              let scope = scopeProp;
-              if (!scope && tablelvl2 && tablelvl2.variant === 'head') {
-                scope = 'col';
-              }
-              const padding = paddingProp || (table && table.padding ? table.padding : 'default');
+            let scope = scopeProp;
+            if (!scope && tablelvl2 && tablelvl2.variant === 'head') {
+              scope = 'col';
+            }
+            const padding = paddingProp || (table && table.padding ? table.padding : 'default');
 
-              const className = classNames(
-                classes.root,
-                {
-                  [classes.head]: variant
-                    ? variant === 'head'
-                    : tablelvl2 && tablelvl2.variant === 'head',
-                  [classes.body]: variant
-                    ? variant === 'body'
-                    : tablelvl2 && tablelvl2.variant === 'body',
-                  [classes.footer]: variant
-                    ? variant === 'footer'
-                    : tablelvl2 && tablelvl2.variant === 'footer',
-                  [classes.numeric]: numeric,
-                  [classes[`padding${capitalize(padding)}`]]: padding !== 'default',
-                },
-                classNameProp,
-              );
+            const className = classNames(
+              classes.root,
+              {
+                [classes.head]: variant
+                  ? variant === 'head'
+                  : tablelvl2 && tablelvl2.variant === 'head',
+                [classes.body]: variant
+                  ? variant === 'body'
+                  : tablelvl2 && tablelvl2.variant === 'body',
+                [classes.footer]: variant
+                  ? variant === 'footer'
+                  : tablelvl2 && tablelvl2.variant === 'footer',
+                [classes.numeric]: numeric,
+                [classes[`padding${capitalize(padding)}`]]: padding !== 'default',
+              },
+              classNameProp,
+            );
 
-              let ariaSort = null;
-              if (sortDirection) {
-                ariaSort = sortDirection === 'asc' ? 'ascending' : 'descending';
-              }
+            let ariaSort = null;
+            if (sortDirection) {
+              ariaSort = sortDirection === 'asc' ? 'ascending' : 'descending';
+            }
 
-              return (
-                <Component className={className} aria-sort={ariaSort} scope={scope} {...other}>
-                  {children}
-                </Component>
-              );
-            }}
-          </Tablelvl2Context.Consumer>
-        );
-      }}
+            return (
+              <Component className={className} aria-sort={ariaSort} scope={scope} {...other}>
+                {children}
+              </Component>
+            );
+          }}
+        </Tablelvl2Context.Consumer>
+      )}
     </TableContext.Consumer>
   );
 }

diff --git a/packages/material-ui/src/TableFooter/TableFooter.js b/packages/material-ui/src/TableFooter/TableFooter.js
@@ -13,10 +13,9 @@ export const styles = {
 
 function TableFooter(props) {
   const { classes, className, component: Component, ...other } = props;
-  const childContext = { variant: 'footer' };
 
   return (
-    <Tablelvl2Context.Provider value={childContext}>
+    <Tablelvl2Context.Provider value={{ variant: 'footer' }}>
       <Component className={classNames(classes.root, className)} {...other} />
     </Tablelvl2Context.Provider>
   );

diff --git a/packages/material-ui/src/TableHead/TableHead.js b/packages/material-ui/src/TableHead/TableHead.js
@@ -13,10 +13,9 @@ export const styles = {
 
 function TableHead(props) {
   const { classes, className, component: Component, ...other } = props;
-  const childContext = { variant: 'head' };
 
   return (
-    <Tablelvl2Context.Provider value={childContext}>
+    <Tablelvl2Context.Provider value={{ variant: 'head' }}>
       <Component className={classNames(classes.root, className)} {...other} />
     </Tablelvl2Context.Provider>
   );