Merge pull request #17 from TedDriggs/show-thumb-with-permanent-track

Show thumb when permanent tracks enabled

Author: xobotyi

src/index.js

@@ -713,15 +713,15 @@ export default class Scrollbar extends React.Component
         if ((permanentScrollbars || permanentScrollbarY)) {
             trackVerticalStyles.display = null;
 
-            if (noScroll || !scrollY) {
+            if (noScroll || noScrollY) {
                 thumbVerticalStyles.display = "none";
             }
         }
 
         if ((permanentScrollbars || permanentScrollbarX)) {
             trackHorizontalStyles.display = null;
 
-            if (noScroll || !scrollX) {
+            if (noScroll || noScrollX) {
                 thumbHorizontalStyles.display = "none";
             }
         }

tests/Scrollbar/rendering.js

@@ -343,6 +343,23 @@ export default function performTests() {
             });
         });
 
+        describe("when always showing scrollbars", () => {
+            it("should render thumb when content taller than viewport", (done) => {
+                render(<Scrollbar style={{ width: 100, height: 100 }} permanentScrollbarY>
+                        <div style={{width: 100, height: 200}} />
+                    </Scrollbar>,
+                    node,
+                    function() {
+                        setTimeout(() => {
+                            expect(this.trackVertical.style.display).not.toBe("none");
+                            expect(this.thumbVertical.style.display).not.toBe("none");
+                            done();
+                        }, 100);
+                    }
+                );
+            });
+        })
+
         describe("only vertical scroll should be blocked", () => {
             it("if noScrollY={ true } is passed", (done) => {
                 render(<Scrollbar style={ {width: 100, height: 100} } noScrollY={ true }>