Merge pull request #13076 from ryanml/non-sortable-tables-prop

Disables Action column header in ledger payments table
brave · Feb 9, 2018 · 0129cea · 0129cea
1 parent b545d61
commit 0129cea
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 1 deletion.
diff --git a/app/renderer/components/common/sortableTable.js b/app/renderer/components/common/sortableTable.js
@@ -479,7 +479,8 @@ class SortableTable extends React.Component {
               [css(styles.table__th)]: true
             }
             const isString = typeof heading === 'string'
-            const sortMethod = this.sortingDisabled ? 'none' : (dataType === 'number' ? 'number' : undefined)
+            const nonSortableColumns = this.props.nonSortableColumns ? this.props.nonSortableColumns : []
+            const sortMethod = (this.sortingDisabled || nonSortableColumns.includes(heading)) ? 'none' : (dataType === 'number' ? 'number' : undefined)
             if (isString) headerClasses['heading-' + heading] = true
             return <th className={cx(headerClasses)}
               data-sort-method={sortMethod}

diff --git a/app/renderer/components/preferences/payment/ledgerTable.js b/app/renderer/components/preferences/payment/ledgerTable.js
@@ -287,6 +287,7 @@ class LedgerTable extends ImmutableComponent {
         fillAvailable
         smallRow
         headings={['', 'publisher', 'include', 'views', 'timeSpent', 'percentage', 'actions']}
+        nonSortableColumns={['actions']}
         defaultHeading='percentage'
         defaultHeadingSortOrder='desc'
         headerClassNames={css(styles.tableTh)}