add helmet instead of updating document.title (#301)

This commit is contained in:
Guy Ben-Aharon
2024-11-03 11:53:53 +02:00
committed by GitHub
parent bbced225b1
commit 3714ca58ea
10 changed files with 388 additions and 338 deletions

View File

@@ -1,4 +1,4 @@
import React, { useCallback, useEffect, useMemo, useState } from 'react';
import React, { useCallback, useMemo, useState } from 'react';
import type { DBTable } from '@/lib/domain/db-table';
import { deepCopy, generateId } from '@/lib/utils';
import { randomColor } from '@/lib/colors';
@@ -28,11 +28,10 @@ import { storageInitialValue } from '../storage-context/storage-context';
export interface ChartDBProviderProps {
diagram?: Diagram;
readonly?: boolean;
skipTitleUpdate?: boolean;
}
export const ChartDBProvider: React.FC<
React.PropsWithChildren<ChartDBProviderProps>
> = ({ children, diagram, readonly, skipTitleUpdate }) => {
> = ({ children, diagram, readonly }) => {
let db = useStorage();
const events = useEventEmitter<ChartDBEvent>();
const navigate = useNavigate();
@@ -64,19 +63,6 @@ export const ChartDBProvider: React.FC<
db = storageInitialValue;
}
useEffect(() => {
if (skipTitleUpdate) {
return;
}
if (diagramName) {
document.title = `ChartDB - ${diagramName} Diagram | Visualize Database Schemas`;
} else {
document.title =
'ChartDB - Create & Visualize Database Schema Diagrams';
}
}, [diagramName, skipTitleUpdate]);
const schemas = useMemo(
() =>
databasesWithSchemas.includes(databaseType)