Cleanup

Author: compulim

README.md

@@ -45,9 +45,10 @@ export default props =>
 | `atEnd` | `boolean` | `true` if the panel is currently near the end (see `mode` and `threshold` |
 | `atTop` | `boolean` | `true` if the panel is currently near top (see `threshold`) |
 | `mode` | `string` | `"bottom"` for scroll-to-bottom, `"top"` for scroll-to-top |
-| `scrollToBottom` | `function` | Call to scroll panel to bottom |
-| `scrollToEnd` | `function` | Call to scroll panel to end |
-| `scrollToTop` | `function` | Call to scroll panel to top |
+| `scrollTo` | `(scrollTop: number) => void` | Scroll panel to specified position |
+| `scrollToBottom` | `() => void` | Scroll panel to bottom |
+| `scrollToEnd` | `() => void` | Scroll panel to end |
+| `scrollToTop` | `() => void` | Scroll panel to top |
 | `threshold` | `number` | Threshold in pixels to consider the panel is near top/bottom, read-only and only set thru `props` |
 
 # Contributions

packages/component/src/ScrollToBottom/Composer.js

@@ -18,13 +18,10 @@ export default class ScrollToBottomComposer extends React.Component {
       atTop: true,
       mode: props.mode,
       handleUpdate: () => this.state.atEnd && this.state.scrollToEnd(),
-      scrollToBottom: () => this.setState(state => ({
-        scrollTop: state.target && (state.target.scrollHeight - state.target.offsetHeight)
-      })),
+      scrollTo: scrollTop => this.setState(() => ({ scrollTop })),
+      scrollToBottom: () => this.state.scrollTo(this.state.target && (this.state.target.scrollHeight - this.state.target.offsetHeight)),
       scrollToEnd: () => this.state.mode === 'top' ? this.state.scrollToTop() : this.state.scrollToBottom(),
-      scrollToTop: () => this.setState(state => ({
-        scrollTop: 0
-      })),
+      scrollToTop: () => this.state.scrollTo(0),
       scrollTop: null,
       setTarget: target => this.setState(() => ({ target })),
       target: null,

packages/component/src/ScrollToBottom/Context.js

@@ -6,6 +6,7 @@ export default React.createContext({
   atTop: true,
   mode: 'bottom',
   handleUpdate: () => 0,
+  scrollTo: () => 0,
   scrollToBottom: () => 0,
   scrollToEnd: () => 0,
   scrollToTop: () => 0,