Split Table._setScrollbarWidth by adding public function Table.getScrollbarWidth (bvaughn#1411)

RoosDaniel · wuweiweiwu · commit a390b2aa3652 · 2019-09-04T21:49:28.000-10:00
* Create public method Table.getScrollbarWidth

* Add documentation for Table.getScrollbarWidth
diff --git a/docs/Table.md b/docs/Table.md
@@ -61,6 +61,10 @@ This may be appropriate if the underlying row data has changed but the row sizes
 
 Gets offset for a given row and alignment.
 
+##### getScrollbarWidth
+
+Gets the scrollbar width used to pad the table-header.
+
 ##### measureAllRows
 Pre-measure all rows in a `Table`.
 
diff --git a/source/Table/Table.js b/source/Table/Table.js
@@ -337,6 +337,17 @@ export default class Table extends React.PureComponent {
     }
   }
 
+  getScrollbarWidth() {
+    if (this.Grid) {
+      const Grid = findDOMNode(this.Grid);
+      const clientWidth = Grid.clientWidth || 0;
+      const offsetWidth = Grid.offsetWidth || 0;
+      return offsetWidth - clientWidth;
+    }
+
+    return 0;
+  }
+
   componentDidMount() {
     this._setScrollbarWidth();
   }
@@ -726,13 +737,8 @@ export default class Table extends React.PureComponent {
   }
 
   _setScrollbarWidth() {
-    if (this.Grid) {
-      const Grid = findDOMNode(this.Grid);
-      const clientWidth = Grid.clientWidth || 0;
-      const offsetWidth = Grid.offsetWidth || 0;
-      const scrollbarWidth = offsetWidth - clientWidth;
+    const scrollbarWidth = this.getScrollbarWidth();
 
-      this.setState({scrollbarWidth});
-    }
+    this.setState({scrollbarWidth});
   }
 }

Original file line number	Diff line number	Diff line change
`@@ -337,6 +337,17 @@ export default class Table extends React.PureComponent {`
`337`	`337`	`}`
`338`	`338`	`}`
`339`	`339`
	`340`	`+ getScrollbarWidth() {`
	`341`	`+ if (this.Grid) {`
	`342`	`+ const Grid = findDOMNode(this.Grid);`
	`343`	`+ const clientWidth = Grid.clientWidth \|\| 0;`
	`344`	`+ const offsetWidth = Grid.offsetWidth \|\| 0;`
	`345`	`+ return offsetWidth - clientWidth;`
	`346`	`+ }`
	`347`	`+`
	`348`	`+ return 0;`
	`349`	`+ }`
	`350`	`+`
`340`	`351`	`componentDidMount() {`
`341`	`352`	`this._setScrollbarWidth();`
`342`	`353`	`}`
`@@ -726,13 +737,8 @@ export default class Table extends React.PureComponent {`
`726`	`737`	`}`
`727`	`738`
`728`	`739`	`_setScrollbarWidth() {`
`729`		`- if (this.Grid) {`
`730`		`- const Grid = findDOMNode(this.Grid);`
`731`		`- const clientWidth = Grid.clientWidth \|\| 0;`
`732`		`- const offsetWidth = Grid.offsetWidth \|\| 0;`
`733`		`- const scrollbarWidth = offsetWidth - clientWidth;`
	`740`	`+ const scrollbarWidth = this.getScrollbarWidth();`
`734`	`741`
`735`		`- this.setState({scrollbarWidth});`
`736`		`- }`
	`742`	`+ this.setState({scrollbarWidth});`
`737`	`743`	`}`
`738`	`744`	`}`