wip

deephaven · Mar 5, 2025 · 764c89f · 764c89f
1 parent d44370b
commit 764c89f
Show file tree

Hide file tree

Showing 2 changed files with 17 additions and 0 deletions.
diff --git a/packages/chart/src/FigureChartModel.test.ts b/packages/chart/src/FigureChartModel.test.ts
@@ -140,6 +140,21 @@ it('handles colors on bar charts properly', () => {
   ]);
 });
 
+it('updates the title correctly', () => {
+  const figure = chartTestUtils.makeFigure();
+  const model = new FigureChartModel(dh, figure);
+  const mockSubscribe = jest.fn();
+  model.subscribe(mockSubscribe);
+
+  model.setTitle('New Title');
+  expect(mockSubscribe).toHaveBeenCalledTimes(1);
+  expect(mockSubscribe).toHaveBeenCalledWith(
+    expect.objectContaining({
+      type: FigureChartModel.EVENT_LAYOUT_UPDATED,
+    })
+  );
+});
+
 describe('axis transform tests', () => {
   it('handles log x-axis properly', () => {
     const xAxis = chartTestUtils.makeAxis({

diff --git a/packages/chart/src/FigureChartModel.ts b/packages/chart/src/FigureChartModel.ts
@@ -625,6 +625,8 @@ class FigureChartModel extends ChartModel {
         ChartUtils.DEFAULT_TITLE_PADDING.t +
         subtitleCount * ChartUtils.SUBTITLE_LINE_HEIGHT * 0.5;
     }
+
+    this.fireLayoutUpdated({ title: this.layout.title });
   }
 
   getPlotWidth(): number {