feat(chart): 增加炮号并优化 Tooltip 展示

新增 WaveformSeries.shotNo 字段，Tooltip 展示炮号及坐标信息，并同步示例与测试。
2026-08-17 10:14:44 +08:00
parent 54477ecbbd
commit 3692e21dbc
14 changed files with 110 additions and 64 deletions
--- a/src/components/core/types.ts
+++ b/src/components/core/types.ts
@@ -13,6 +13,7 @@ import type { NormalizedWaveformGridLineOptions } from './grid'
 */
 export interface DisplaySeries {
  id: string
+  shotNo?: string
  trackId?: string
  name: string
  unit?: string
@@ -63,6 +64,7 @@ export interface WaveformYAxisLayout {
 */
 export interface HoveredSeriesPoint {
  id: string
+  shotNo?: string
  name: string
  unit?: string
  color: string
--- a/src/components/interaction/WaveformTooltip.test.ts
+++ b/src/components/interaction/WaveformTooltip.test.ts
@@ -80,7 +80,7 @@ describe('WaveformTooltip', () => {
    expect(tooltip.attributes('style')).toContain('max-width: 280px')
  })

-  it('shows resolved asymmetric errors beside the hovered value', () => {
+  it('omits units and errors from the strict x/y value format', () => {
    const pointWithErrors = { x: 1, y: 12, error: 1, upperError: 2 }
    const wrapper = mount(WaveformTooltip, {
      props: {
@@ -102,10 +102,12 @@ describe('WaveformTooltip', () => {
      },
    })

-    expect(wrapper.get('.waveform-tooltip__series small').text()).toBe('(+2 / -1)')
+    expect(wrapper.get('.waveform-tooltip__value').text()).toBe('(x:1 y:12)')
+    expect(wrapper.get('.waveform-tooltip__value').text()).not.toContain('C')
+    expect(wrapper.get('.waveform-tooltip__value').text()).not.toContain('+2')
  })

-  it('keeps a formatted value and its unit in one value container', () => {
+  it('renders the configured shot number, channel, and formatted coordinates', () => {
    const hoveredPoint = { x: 1, y: -1405.4932 }
    const wrapper = mount(WaveformTooltip, {
      props: {
@@ -117,6 +119,7 @@ describe('WaveformTooltip', () => {
          {
            trackIndex: 0,
            name: 'ENG8KJXAc(10001)',
+            shotNo: '炮 7',
            color: '#ffb43b',
            unit: 'A',
            point: hoveredPoint,
@@ -127,26 +130,31 @@ describe('WaveformTooltip', () => {
      },
    })

-    expect(wrapper.get('.waveform-tooltip__value').text()).toBe('-1,405.4932 A')
+    expect(wrapper.get('.waveform-tooltip__series-label').text()).toBe('炮 7： ENG8KJXAc(10001)(A)')
+    expect(wrapper.get('.waveform-tooltip__value').text()).toBe('(x:1,000 y:-1,405.4932)')
  })

-  it('formats tooltip time with at most four decimal places', () => {
+  it('renders each series row with the formatted x coordinate and series label', () => {
    const wrapper = mount(WaveformTooltip, {
      props: {
        visible: true,
        position: { x: 10, y: 10 },
        timeUnit: 's',
        hoveredPoint: { x: 1.234567, y: 12 },
-        seriesPoints: [],
+        seriesPoints: [
+          { trackIndex: 0, name: '温度', color: '#f00', point: { x: 1.234567, y: 12 } },
+        ],
        containerWidth: 400,
        containerHeight: 300,
      },
    })

-    expect(wrapper.get('.waveform-tooltip__time').text()).toBe('s: 1.2346')
+    expect(wrapper.find('.waveform-tooltip__time').exists()).toBe(false)
+    expect(wrapper.get('.waveform-tooltip__series-label').text()).toBe('未配置炮号： 温度')
+    expect(wrapper.get('.waveform-tooltip__value').text()).toBe('(x:1.2346 y:12)')
  })

-  it('omits the error label when both resolved errors are zero', () => {
+  it('uses the fallback shot number when shotNo is blank', () => {
    const point = { x: 1, y: 12 }
    const wrapper = mount(WaveformTooltip, {
      props: {
@@ -160,6 +168,6 @@ describe('WaveformTooltip', () => {
      },
    })

-    expect(wrapper.find('.waveform-tooltip__series small').exists()).toBe(false)
+    expect(wrapper.get('.waveform-tooltip__series-label').text()).toBe('未配置炮号： 温度')
  })
 })
--- a/src/components/interaction/WaveformTooltip.vue
+++ b/src/components/interaction/WaveformTooltip.vue
@@ -1,12 +1,12 @@
 <script setup lang="ts">
 import { computed } from 'vue'
-import { resolveWaveformPointErrors } from '../../core'
 import { formatTooltipNumber, formatTooltipTime } from '../../utils'
 import type { WaveformPoint } from '../data/types'

 interface SeriesPoint {
  trackIndex: number
  name: string
+  shotNo?: string
  color: string
  unit?: string
  point: WaveformPoint
@@ -34,7 +34,7 @@ const props = defineProps<Props>()
 const tooltipGap = 12
 const containerPadding = 8
 const tooltipPlacementWidth = 238
-const tooltipMaxWidth = 320
+const tooltipMaxWidth = 560
 const tooltipHorizontalPadding = 20
 const tooltipLineHeight = 20

@@ -45,18 +45,18 @@ function estimateLineCount(text: string, width: number): number {
 }

 function formatSeriesText(seriesPoint: SeriesPoint): string {
-  const error = formatError(seriesPoint.point)
-  return `${seriesPoint.name ? `${seriesPoint.name}: ` : ''}${formatTooltipNumber(seriesPoint.point.y)}${
-    seriesPoint.unit ? ` ${seriesPoint.unit}` : ''
-  }${error ? ` ${error}` : ''}`
+  const timeText = formatTooltipTime(props.hoveredPoint?.x ?? seriesPoint.point.x, props.timeUnit)
+  return `${seriesPoint.shotNo?.trim() || '未配置炮号'}： ${seriesPoint.name}${
+    seriesPoint.unit ? `(${seriesPoint.unit})` : ''
+  }  (x:${timeText} y:${formatTooltipNumber(seriesPoint.point.y)})`
 }

-function estimateTooltipHeight(width: number, timeText: string): number {
+function estimateTooltipHeight(width: number): number {
  const seriesLines = props.seriesPoints.reduce(
    (total, seriesPoint) => total + estimateLineCount(formatSeriesText(seriesPoint), width),
    0,
  )
-  return 16 + tooltipLineHeight * (estimateLineCount(timeText, width) + seriesLines) + 5
+  return 16 + tooltipLineHeight * seriesLines + 5
 }

 const tooltipStyle = computed(() => {
@@ -84,25 +84,18 @@ const tooltipStyle = computed(() => {
        : { left: `${containerPadding}px`, maxWidth: `${availableWidth}px` }

  const maxWidth = Number.parseFloat(horizontalStyle.maxWidth)
-  const timeText = `${props.timeUnit}: ${formatTooltipTime(props.hoveredPoint.x, props.timeUnit)}`
-
  return {
    ...horizontalStyle,
    top: `${Math.max(
      8,
      Math.min(
        props.position.y - 18,
-        props.containerHeight - estimateTooltipHeight(maxWidth, timeText) - 8,
+        props.containerHeight - estimateTooltipHeight(maxWidth) - 8,
      ),
    )}px`,
  }
 })

-function formatError(point: WaveformPoint): string | null {
-  const { lower, upper } = resolveWaveformPointErrors(point)
-  if (lower === 0 && upper === 0) return null
-  return `(+${formatTooltipNumber(upper)} / -${formatTooltipNumber(lower)})`
-}
 </script>

 <template>
@@ -111,9 +104,6 @@ function formatError(point: WaveformPoint): string | null {
    class="waveform-tooltip waveform-chart__tooltip"
    :style="tooltipStyle"
  >
-    <span class="waveform-tooltip__time waveform-chart__tooltip-time">
-      {{ timeUnit }}: {{ formatTooltipTime(hoveredPoint.x, timeUnit) }}
-    </span>
    <span
      v-for="seriesPoint in seriesPoints"
      :key="`${seriesPoint.trackIndex}-${seriesPoint.name}`"
@@ -121,12 +111,12 @@ function formatError(point: WaveformPoint): string | null {
    >
      <i :style="{ backgroundColor: seriesPoint.color }" />
      <span class="waveform-tooltip__series-content">
-        <strong v-if="seriesPoint.name">{{ seriesPoint.name }}:</strong>
-        <span class="waveform-tooltip__value">
-          {{ formatTooltipNumber(seriesPoint.point.y)
-          }}{{ seriesPoint.unit ? ` ${seriesPoint.unit}` : '' }}
-          <small v-if="formatError(seriesPoint.point)">{{ formatError(seriesPoint.point) }}</small>
-        </span>
+        <span class="waveform-tooltip__series-label">{{ seriesPoint.shotNo?.trim() || '未配置炮号' }}： {{
+          seriesPoint.name
+        }}<template v-if="seriesPoint.unit">({{ seriesPoint.unit }})</template></span>
+        <span class="waveform-tooltip__value">(x:{{ formatTooltipTime(hoveredPoint.x, timeUnit) }} y:{{
+          formatTooltipNumber(seriesPoint.point.y)
+        }})</span>
      </span>
    </span>
  </div>
@@ -138,26 +128,17 @@ function formatError(point: WaveformPoint): string | null {
  position: absolute;
  z-index: 2;
  display: grid;
-  gap: 3px;
+  gap: 2px;
  width: max-content;
-  max-width: min(320px, calc(100% - 16px));
-  padding: 8px 10px;
-  color: #333;
-  font:
-    12px/1.45 ui-monospace,
-    SFMono-Regular,
-    Consolas,
-    monospace;
+  max-width: min(560px, calc(100% - 16px));
+  padding: 9px 12px;
+  color: #505050;
+  font: 14px/1.35 Arial, sans-serif;
  pointer-events: none;
  background: #fff;
-  border: 1px solid #d9d9d9;
-  border-radius: 2px;
-  box-shadow: 0 4px 12px rgb(16 24 40 / 12%);
-}
-
-.waveform-tooltip__time {
-  padding-bottom: 4px;
-  border-bottom: 1px solid #f0f0f0;
+  border: 1px solid #e3e7eb;
+  border-radius: 4px;
+  box-shadow: 0 3px 10px rgb(16 24 40 / 15%);
 }

 .waveform-tooltip__series {
@@ -176,9 +157,24 @@ function formatError(point: WaveformPoint): string | null {
 }

 .waveform-tooltip__series-content {
+  display: flex;
+  flex-wrap: nowrap;
+  gap: 12px;
+  align-items: baseline;
  min-width: 0;
-  overflow-wrap: anywhere;
-  white-space: normal;
+}
+
+.waveform-tooltip__series-label {
+  flex: 1 1 auto;
+  min-width: 0;
+  overflow: hidden;
+  text-overflow: ellipsis;
+  white-space: nowrap;
+}
+
+.waveform-tooltip__value {
+  flex: 0 0 auto;
+  white-space: nowrap;
 }

 .waveform-tooltip__series strong {
@@ -186,7 +182,7 @@ function formatError(point: WaveformPoint): string | null {
 }

 .waveform-tooltip__value {
-  overflow-wrap: anywhere;
+  color: #555;
 }

 .waveform-tooltip__series small {
--- a/src/components/interaction/useWaveformHover.ts
+++ b/src/components/interaction/useWaveformHover.ts
@@ -85,6 +85,7 @@ export function useWaveformHover(context: HoverContext) {
    point: WaveformPoint,
  ): HoveredSeriesPoint => ({
    id: series.id,
+    shotNo: series.shotNo,
    name: series.name,
    color: series.color,
    unit: series.unit,
--- a/src/components/waveformChartCases/compactAndSharedHover.test.ts
+++ b/src/components/waveformChartCases/compactAndSharedHover.test.ts
@@ -179,10 +179,10 @@ describe('WaveformChart', () => {
    await flushPromises()

    const tooltip = wrapper.get('.waveform-chart__tooltip')
-    expect(tooltip.text()).toContain('BT2_2M:')
-    expect(tooltip.text()).toContain('2 T')
-    expect(tooltip.text()).toContain('BT1_2M:')
-    expect(tooltip.text()).toContain('4 T')
+    expect(tooltip.text()).toContain('未配置炮号： BT2_2M')
+    expect(tooltip.text()).toContain('(x:1,000 y:2)')
+    expect(tooltip.text()).toContain('未配置炮号： BT1_2M')
+    expect(tooltip.text()).toContain('(x:1,000 y:4)')
    const crosshairLines = wrapper.findAll('.waveform-chart__crosshair line')
    expect(crosshairLines).toHaveLength(2)
    crosshairLines.forEach((line) => {
--- a/src/components/waveformChartCases/dataAndHoverPerformance.test.ts
+++ b/src/components/waveformChartCases/dataAndHoverPerformance.test.ts
@@ -178,7 +178,7 @@ describe('WaveformChart', () => {
    await flushPromises()
    expect(wrapper.emitted('point-hover')?.at(-1)).toEqual([{ x: 1, y: 5 }])
    expect(wrapper.find('.waveform-chart__tooltip').exists()).toBe(true)
-    expect(wrapper.get('.waveform-chart__tooltip-time').text()).toBe('ms: 1,000')
+    expect(wrapper.get('.waveform-tooltip__series-label').text()).toContain('未配置炮号：')
    const crosshairLines = wrapper.findAll('.waveform-chart__crosshair line')
    expect(crosshairLines).toHaveLength(1)
    expect(crosshairLines[0].attributes('x1')).toBe(crosshairLines[0].attributes('x2'))
@@ -303,7 +303,7 @@ describe('WaveformChart', () => {
    flushAnimationFrames()
    await flushPromises()

-    expect(wrapper.get('.waveform-chart__tooltip-time').text()).toBe('ms: 1,000')
+    expect(wrapper.get('.waveform-tooltip__series-label').text()).toContain('未配置炮号：')
    expect(wrapper.findAll('.waveform-chart__crosshair line')).toHaveLength(1)
    expect(wrapper.get('.waveform-chart__line').element).toBe(pathBeforeHover)
    expect(chartUpdate).not.toHaveBeenCalled()