Include Elasticsearch node into screen reader message, per review fee…

…dback
elastic · Dec 27, 2019 · 9e8b05e · 9e8b05e
1 parent 8cb71f0
commit 9e8b05e
Showing 1 changed file with 6 additions and 3 deletions.
diff --git a/x-pack/legacy/plugins/monitoring/public/components/elasticsearch/node/node.js b/x-pack/legacy/plugins/monitoring/public/components/elasticsearch/node/node.js
@@ -19,7 +19,11 @@ import { NodeDetailStatus } from '../node_detail_status';
 import { Logs } from '../../logs/';
 import { MonitoringTimeseriesContainer } from '../../chart';
 import { ShardAllocation } from '../shard_allocation/shard_allocation';
-import { FormattedMessage  } from '@kbn/i18n/react';
+import { FormattedMessage } from '@kbn/i18n/react';
+
+function formatScreenReaderMsg(msg, nodeSummary) {
+  return msg + ' ' + nodeSummary.name;
+}
 
 export const Node = ({
   nodeSummary,
@@ -40,15 +44,14 @@ export const Node = ({
     metrics.node_latency,
     metrics.node_segment_count,
   ];
-
   return (
     <EuiPage>
       <EuiPageBody>
         <EuiScreenReaderOnly>
           <h1>
             <FormattedMessage
               id="xpack.monitoring.elasticsearch.node.heading"
-              defaultMessage="Elasticsearch node"
+              defaultMessage={formatScreenReaderMsg('Elasticsearch node', nodeSummary)}
             />
           </h1>
         </EuiScreenReaderOnly>