github-mirrors/kibana
1
0
Fork 0
mirror of https://github.com/elastic/kibana.git synced 2025-04-24 09:48:58 -04:00
Code Issues Projects Releases Packages Wiki Activity

[Uptime] Disable sorting on EuiInMemoryTable instances (#30430) (#31141)

Browse source 
All these tables are for large datasets, which makes the sorting misleading. After we have the states index we can query things in such a way that we can enable sorting (and pagination!). Till then we should disable these to better adhere to the principle of least surprise.
This commit is contained in:
Andrew Cholakian 2019-02-14 11:40:20 -06:00 • committed by GitHub
parent 23089a5315
commit 9c5d95ecd7
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
6 changed files with 3 additions and 30 deletions
Download patch file Download diff file Expand all files Collapse all files

8
x-pack/plugins/uptime/public/components/functional/__tests__/__snapshots__/error_list.test.tsx.snap
View file

@ -25,35 +25,29 @@ exports[`ErrorList component renders the error list without errors 1`] = `
Object {
"field": "type",
"name": "Error type",
"sortable": true,
},
Object {
"field": "monitorId",
"name": "Monitor ID",
"render": [Function],
"sortable": true,
"width": "25%",
},
Object {
"field": "count",
"name": "Count",
"sortable": true,
},
Object {
"field": "timestamp",
"name": "Latest error",
"render": [Function],
"sortable": true,
},
Object {
"field": "statusCode",
"name": "Status code",
"sortable": true,
},
Object {
"field": "latestMessage",
"name": "Latest message",
"sortable": true,
"width": "40%",
},
]
@ -108,7 +102,7 @@ exports[`ErrorList component renders the error list without errors 1`] = `
}
}
responsive={true}
sorting={true}
sorting={false}
/>
</EuiPanel>
</Fragment>

5
x-pack/plugins/uptime/public/components/functional/__tests__/__snapshots__/monitor_list.test.tsx.snap
View file

@ -26,13 +26,11 @@ exports[`MonitorList component renders a monitor list without errors 1`] = `
"field": "ping.monitor.status",
"name": "Status",
"render": [Function],
"sortable": true,
},
Object {
"field": "ping.timestamp",
"name": "Last updated",
"render": [Function],
"sortable": true,
},
Object {
"field": "ping.monitor.id",
@ -47,7 +45,6 @@ exports[`MonitorList component renders a monitor list without errors 1`] = `
Object {
"field": "ping.monitor.ip",
"name": "IP",
"sortable": true,
},
Object {
"field": "upSeries",
@ -1284,7 +1281,7 @@ exports[`MonitorList component renders a monitor list without errors 1`] = `
}
}
responsive={true}
sorting={true}
sorting={false}
/>
</EuiPanel>
</Fragment>

5
x-pack/plugins/uptime/public/components/functional/__tests__/__snapshots__/ping_list.test.tsx.snap
View file

@ -101,13 +101,11 @@ exports[`PingList component renders sorted list without errors 1`] = `
"field": "monitor.status",
"name": "Status",
"render": [Function],
"sortable": true,
},
Object {
"field": "timestamp",
"name": "Timestamp",
"render": [Function],
"sortable": true,
},
Object {
"field": "monitor.ip",
@ -123,7 +121,6 @@ exports[`PingList component renders sorted list without errors 1`] = `
"field": "monitor.duration.us",
"name": "Duration ms",
"render": [Function],
"sortable": true,
},
Object {
"field": "error.type",
@ -347,7 +344,7 @@ exports[`PingList component renders sorted list without errors 1`] = `
}
}
responsive={true}
sorting={true}
sorting={false}
/>
</EuiPanel>
</Fragment>

7
x-pack/plugins/uptime/public/components/functional/error_list.tsx
View file

@ -35,7 +35,6 @@ export const ErrorList = ({ loading, errorList }: ErrorListProps) => (
name: i18n.translate('xpack.uptime.errorList.errorTypeColumnLabel', {
defaultMessage: 'Error type',
}),
sortable: true,
},
{
field: 'monitorId',
@ -43,7 +42,6 @@ export const ErrorList = ({ loading, errorList }: ErrorListProps) => (
defaultMessage: 'Monitor ID',
}),
render: (id: string) => <Link to={`/monitor/${id}`}>{id}</Link>,
sortable: true,
width: '25%',
},
{
@ -51,14 +49,12 @@ export const ErrorList = ({ loading, errorList }: ErrorListProps) => (
name: i18n.translate('xpack.uptime.errorList.CountColumnLabel', {
defaultMessage: 'Count',
}),
sortable: true,
},
{
field: 'timestamp',
name: i18n.translate('xpack.uptime.errorList.latestErrorColumnLabel', {
defaultMessage: 'Latest error',
}),
sortable: true,
render: (timestamp: string) => moment(timestamp).fromNow(),
},
{
@ -66,18 +62,15 @@ export const ErrorList = ({ loading, errorList }: ErrorListProps) => (
name: i18n.translate('xpack.uptime.errorList.statusCodeColumnLabel', {
defaultMessage: 'Status code',
}),
sortable: true,
},
{
field: 'latestMessage',
name: i18n.translate('xpack.uptime.errorList.latestMessageColumnLabel', {
defaultMessage: 'Latest message',
}),
sortable: true,
width: '40%',
},
]}
sorting={true}
pagination={{ initialPageSize: 10, pageSizeOptions: [5, 10, 20, 50] }}
/>
</EuiPanel>

4
x-pack/plugins/uptime/public/components/functional/monitor_list.tsx
View file

@ -69,7 +69,6 @@ export const MonitorList = ({ dangerColor, loading, monitors, primaryColor }: Mo
})}
</EuiHealth>
),
sortable: true,
},
{
field: 'ping.timestamp',
@ -77,7 +76,6 @@ export const MonitorList = ({ dangerColor, loading, monitors, primaryColor }: Mo
defaultMessage: 'Last updated',
}),
render: (timestamp: string) => moment(timestamp).fromNow(),
sortable: true,
},
{
field: 'ping.monitor.id',
@ -108,7 +106,6 @@ export const MonitorList = ({ dangerColor, loading, monitors, primaryColor }: Mo
name: i18n.translate('xpack.uptime.monitorList.ipColumnLabel', {
defaultMessage: 'IP',
}),
sortable: true,
},
{
field: 'upSeries',
@ -151,7 +148,6 @@ export const MonitorList = ({ dangerColor, loading, monitors, primaryColor }: Mo
loading={loading}
items={monitors}
pagination={monitorListPagination}
sorting={true}
/>
</EuiPanel>
</Fragment>

4
x-pack/plugins/uptime/public/components/functional/ping_list.tsx
View file

@ -60,14 +60,12 @@ export const PingList = ({
})}
</EuiHealth>
),
sortable: true,
},
{
field: 'timestamp',
name: i18n.translate('xpack.uptime.pingList.timestampColumnLabel', {
defaultMessage: 'Timestamp',
}),
sortable: true,
render: (timestamp: string) => moment(timestamp).fromNow(),
},
{
@ -91,7 +89,6 @@ export const PingList = ({
description: 'The "ms" in the default message is an abbreviation for milliseconds',
}),
render: (duration: number) => duration / 1000,
sortable: true,
},
{
field: 'error.type',
@ -182,7 +179,6 @@ export const PingList = ({
columns={columns}
items={pings}
pagination={{ initialPageSize: 10, pageSizeOptions: [5, 10, 20, 100] }}
sorting={true}
/>
</EuiPanel>
</Fragment>