From 6e55cb4269d20eb2cdb0109bf48737a78292adc9 Mon Sep 17 00:00:00 2001
From: Peter Shin <peterjs@fb.com>
Date: Fri, 4 Mar 2016 09:13:59 -0800
Subject: [PATCH] Add not allowed styles to bulk checkbox.

---
 components/DataBrowserHeaderBar/DataBrowserHeaderBar.react.js | 4 +++-
 components/DataBrowserHeaderBar/DataBrowserHeaderBar.scss     | 4 ++++
 2 files changed, 7 insertions(+), 1 deletion(-)

diff --git a/components/DataBrowserHeaderBar/DataBrowserHeaderBar.react.js b/components/DataBrowserHeaderBar/DataBrowserHeaderBar.react.js
index 3030c07b2c..fc2c2aa605 100644
--- a/components/DataBrowserHeaderBar/DataBrowserHeaderBar.react.js
+++ b/components/DataBrowserHeaderBar/DataBrowserHeaderBar.react.js
@@ -17,8 +17,10 @@ export default class DataBrowserHeaderBar extends React.Component {
   render() {
     let { headers, onResize, selected, selectAll, onAddColumn, updateOrdering, readonly, handleDragDrop } = this.props;
     let elements = [
+      // Note: bulk checkbox is disabled as all rows are selected (not just visible ones due to current lazy loading implementation)
+      // TODO: add bulk checking only visible rows
       <div key='check' className={[styles.wrap, styles.check].join(' ')}>
-        {readonly ? null : <input type='checkbox' disabled={true} checked={false} onChange={(e) => selectAll(e.target.checked)} />}
+        {readonly ? null : <input className={styles.disabled} type='checkbox' disabled={true} checked={false} onChange={(e) => selectAll(e.target.checked)} />}
       </div>
     ];
 
diff --git a/components/DataBrowserHeaderBar/DataBrowserHeaderBar.scss b/components/DataBrowserHeaderBar/DataBrowserHeaderBar.scss
index 2f21ff4c88..be4c3dc317 100644
--- a/components/DataBrowserHeaderBar/DataBrowserHeaderBar.scss
+++ b/components/DataBrowserHeaderBar/DataBrowserHeaderBar.scss
@@ -64,3 +64,7 @@
   margin: 0 -4px;
   cursor: ew-resize;
 }
+
+.disabled {
+  cursor: not-allowed;
+}