mirror of https://github.com/grafana/grafana.git
				
				
				
			Barchart: Apply field visibility to traceID (#68524)
This commit is contained in:
		
							parent
							
								
									3ffbd2a1db
								
							
						
					
					
						commit
						160e207e47
					
				| 
						 | 
					@ -40,10 +40,14 @@ export const DataHoverView = ({ data, rowIndex, columnIndex, sortOrder, mode, he
 | 
				
			||||||
  const fields = data.fields.map((f, idx) => {
 | 
					  const fields = data.fields.map((f, idx) => {
 | 
				
			||||||
    return { ...f, hovered: idx === columnIndex };
 | 
					    return { ...f, hovered: idx === columnIndex };
 | 
				
			||||||
  });
 | 
					  });
 | 
				
			||||||
  // Put the traceID field in front.
 | 
					 | 
				
			||||||
  const visibleFields = fields.filter((f) => !Boolean(f.config.custom?.hideFrom?.tooltip));
 | 
					  const visibleFields = fields.filter((f) => !Boolean(f.config.custom?.hideFrom?.tooltip));
 | 
				
			||||||
  const traceIDField = visibleFields.find((field) => field.name === 'traceID') || fields[0];
 | 
					  const traceIDField = visibleFields.find((field) => field.name === 'traceID') || fields[0];
 | 
				
			||||||
  const orderedVisibleFields = [traceIDField, ...visibleFields.filter((field) => traceIDField !== field)];
 | 
					  const orderedVisibleFields = [];
 | 
				
			||||||
 | 
					  // Only include traceID if it's visible and put it in front.
 | 
				
			||||||
 | 
					  if (visibleFields.filter((field) => traceIDField === field).length > 0) {
 | 
				
			||||||
 | 
					    orderedVisibleFields.push(traceIDField);
 | 
				
			||||||
 | 
					  }
 | 
				
			||||||
 | 
					  orderedVisibleFields.push(...visibleFields.filter((field) => traceIDField !== field));
 | 
				
			||||||
 | 
					
 | 
				
			||||||
  if (orderedVisibleFields.length === 0) {
 | 
					  if (orderedVisibleFields.length === 0) {
 | 
				
			||||||
    return null;
 | 
					    return null;
 | 
				
			||||||
| 
						 | 
					
 | 
				
			||||||
		Loading…
	
		Reference in New Issue