update: in the app, add missing favicons and use dynamic title (#508)
* update: adds multiple favicon options to the app * update: uses the current workbook name in as page title * update: replace favicons in assets
This commit is contained in:
committed by
GitHub
parent
68a33a5f87
commit
3d265bba27
@@ -75,6 +75,16 @@ function App() {
|
||||
start();
|
||||
}, []);
|
||||
|
||||
// biome-ignore lint/correctness/useExhaustiveDependencies: localStorageId needed to detect name changes (model mutates internally)
|
||||
useEffect(() => {
|
||||
if (model) {
|
||||
const workbookName = model.getName();
|
||||
document.title = workbookName ? `${workbookName} - IronCalc` : "IronCalc";
|
||||
} else {
|
||||
document.title = "IronCalc";
|
||||
}
|
||||
}, [model, localStorageId]);
|
||||
|
||||
if (!model) {
|
||||
return (
|
||||
<Loading>
|
||||
|
||||
Reference in New Issue
Block a user