Skip to content

Commit

Permalink
Add tooltip for group columns and adjust copy to reflect groups in ot…
Browse files Browse the repository at this point in the history
…her tooltips
  • Loading branch information
alvarezmelissa87 committed Sep 23, 2022
1 parent c2b5ec8 commit cafcacb
Showing 1 changed file with 22 additions and 5 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -233,9 +233,26 @@ export const SpikeAnalysisGroupsTable: FC<SpikeAnalysisTableProps> = ({
{
'data-test-subj': 'aiopsSpikeAnalysisGroupsTableColumnGroup',
field: 'group',
name: i18n.translate('xpack.aiops.explainLogRateSpikes.spikeAnalysisTableGroups.groupLabel', {
defaultMessage: 'Group',
}),
name: (
<EuiToolTip
position="top"
content={i18n.translate(
'xpack.aiops.explainLogRateSpikes.spikeAnalysisTableGroups.groupColumnTooltip',
{
defaultMessage:
'Displays field values unique to the group. Expand row to see all values.',
}
)}
>
<>
<FormattedMessage
id="xpack.aiops.explainLogRateSpikes.spikeAnalysisTableGroups.groupLabel"
defaultMessage="Group"
/>
<EuiIcon size="s" color="subdued" type="questionInCircle" className="eui-alignTop" />
</>
</EuiToolTip>
),
render: (_, { group, repeatedValues }) => {
const valuesBadges = [];
for (const fieldName in group) {
Expand Down Expand Up @@ -292,7 +309,7 @@ export const SpikeAnalysisGroupsTable: FC<SpikeAnalysisTableProps> = ({
'xpack.aiops.explainLogRateSpikes.spikeAnalysisTableGroups.logRateColumnTooltip',
{
defaultMessage:
'A visual representation of the impact of the field on the message rate difference',
'A visual representation of the impact of the group on the message rate difference',
}
)}
>
Expand Down Expand Up @@ -368,7 +385,7 @@ export const SpikeAnalysisGroupsTable: FC<SpikeAnalysisTableProps> = ({
content={i18n.translate(
'xpack.aiops.explainLogRateSpikes.spikeAnalysisTable.impactLabelColumnTooltip',
{
defaultMessage: 'The level of impact of the field on the message rate difference',
defaultMessage: 'The level of impact of the group on the message rate difference',
}
)}
>
Expand Down

0 comments on commit cafcacb

Please sign in to comment.