feat(xo-web/host/logs): use SortedTable actions (#3313)

See #3179

CHANGELOG.md

@@ -6,6 +6,7 @@
 
 - [Tables] Filter input now always shows up even if the table is empty [#3295](https://github.com/vatesfr/xen-orchestra/issues/3295) (PR [#3296](https://github.com/vatesfr/xen-orchestra/pull/3296))
 - [Tasks] The table is now still shown when there are no tasks (PR [#3305](https://github.com/vatesfr/xen-orchestra/pull/3305))
+- [Host / Logs] Homogenize action buttons in table and enable bulk deletion [#3179](https://github.com/vatesfr/xen-orchestra/issues/3179) (PR [#3313](https://github.com/vatesfr/xen-orchestra/pull/3313))
 
 ### Bug fixes
 

packages/xo-web/src/xo-app/host/tab-logs.js

@@ -1,15 +1,12 @@
 import _ from 'intl'
-import ActionRowButton from 'action-row-button'
 import Component from 'base-component'
 import isEmpty from 'lodash/isEmpty'
-import map from 'lodash/map'
 import React from 'react'
 import SortedTable from 'sorted-table'
-import TabButton from 'tab-button'
-import { deleteMessage } from 'xo'
 import { createPager } from 'selectors'
+import { Row, Col } from 'grid'
+import { deleteMessage, deleteMessages } from 'xo'
 import { FormattedRelative, FormattedTime } from 'react-intl'
-import { Container, Row, Col } from 'grid'
 
 const LOG_COLUMNS = [
   {
@@ -40,16 +37,16 @@ const LOG_COLUMNS = [
     itemRenderer: log => log.body,
     sortCriteria: log => log.body,
   },
+]
+
+const LOG_ACTIONS = [
   {
-    name: _('logAction'),
+    handler: deleteMessages,
-    itemRenderer: log => (
+    individualHandler: deleteMessage,
-      <ActionRowButton
+    individualLabel: _('logDelete'),
-        btnStyle='danger'
+    icon: 'delete',
-        handler={deleteMessage}
+    label: _('logsDelete'),
-        handlerParam={log}
+    level: 'danger',
-        icon='delete'
-      />
-    ),
   },
 ]
 
@@ -64,7 +61,6 @@ export default class TabLogs extends Component {
     }
   }
 
-  _deleteAllLogs = () => map(this.props.logs, deleteMessage)
   _nextPage = () => this.setState({ page: this.state.page + 1 })
   _previousPage = () => this.setState({ page: this.state.page - 1 })
 
@@ -83,23 +79,11 @@ export default class TabLogs extends Component {
     }
 
     return (
-      <Container>
+      <SortedTable
-        <Row>
+        actions={LOG_ACTIONS}
-          <Col className='text-xs-right'>
+        collection={logs}
-            <TabButton
+        columns={LOG_COLUMNS}
-              btnStyle='danger'
+      />
-              handler={this._deleteAllLogs}
-              icon='delete'
-              labelId='logRemoveAll'
-            />
-          </Col>
-        </Row>
-        <Row>
-          <Col>
-            <SortedTable collection={logs} columns={LOG_COLUMNS} />
-          </Col>
-        </Row>
-      </Container>
     )
   }
 }