Startseite heisst /home, Bearbeiten steht in der URL
Die Adresszeile sagte nicht, wo man ist. Die Startseite lag auf "/", waehrend jede andere Seite einen sprechenden Pfad hat, und der Bearbeitungsmodus des Dashboards stand nirgends: Neuladen warf einen heraus, der Stand liess sich nicht verschicken, die Zurueck-Taste tat nichts. Jetzt /home und /home/edit, fuer weitere Dashboards /d/:id und /d/:id/edit. "/" und jeder unbekannte Pfad leiten sichtbar auf /home um. Der Kern der Arbeit steckte in Dashboard.jsx: Der Modus war nicht als eigenes Flag gefuehrt, sondern aus `entwurf !== null` abgeleitet - eine Variable, die zugleich Modus und ungespeicherte Aenderung war. Der Modus kommt jetzt als Prop aus der Route, `entwurf` ist nur noch das, was sein Name sagt. Weil der Entwurf allein im Speicher lebt, baut ihn ein Effekt aus dem gespeicherten Layout auf, sobald dieses geladen ist - sonst koennte man /home/edit nicht direkt aufrufen oder neu laden. Dabei eine Falle vermieden: `aktives` ist nach jedem setDashboards ein neues Objekt. Ohne Merker, zu welchem Dashboard der Entwurf gehoert, haette jedes Neuladen der Liste waehrend des Bearbeitens die ungespeicherte Anordnung verworfen. "Anordnen" navigiert mit push, damit die Zurueck-Taste den Modus verlaesst; "Speichern" und "Abbrechen" mit replace, damit sie nicht hineinfuehrt. /home/edit bei verbindlicher Admin-Vorgabe faellt auf die Ansicht zurueck, statt eine Leiste ohne Speichern-Knopf zu zeigen. Dazu zwei Dinge, die sonst stillschweigend gebrochen waeren: - Der ErrorBoundary-Schluessel hing am ganzen Pfad. /home/edit haette als andere Seite gegolten, den Baum neu eingehaengt und beim Umschalten Dashboards, Artikel und Lagerorte erneut geladen. Jetzt nur das erste Pfadsegment - im Mitschnitt steht beim Umschalten kein einziger Aufruf. - Die Seitenleisten-Links trugen `end` (exakter Match), weil "/" sonst ueberall als Treffer galt. Bei /home ist das nicht noetig, und ohne `end` bleibt der Eintrag auch unter /home/edit markiert. Von Hand durchgeklickt: "/" und "/gibtsnicht" landen auf /home; Anordnen -> /home/edit, Neuladen bleibt im Modus mit sieben Karten als Entwurf, Zurueck verlaesst ihn, nach Abbrechen fuehrt Zurueck nicht wieder hinein; zweites Dashboard -> /d/1/edit; bei erzwungener Vorgabe faellt /home/edit auf /home zurueck. Backend-Suite unveraendert bei 517. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
This commit is contained in:
co-authored by
Claude Opus 5
parent
98db3339a2
commit
a1b10d9ec3
+25
-8
@@ -108,11 +108,13 @@ function Sidebar() {
|
||||
<BrandMark />
|
||||
</div>
|
||||
<nav className="sidebar-nav">
|
||||
<NavItem to="/" end icon="dashboard" label="Übersicht" />
|
||||
{/* Ohne `end`: Der Eintrag soll auch bei /home/edit markiert bleiben.
|
||||
Bei "/" war `end` noetig, weil sonst JEDER Pfad als Treffer galt. */}
|
||||
<NavItem to="/home" icon="dashboard" label="Übersicht" />
|
||||
{/* Ab dem zweiten Dashboard lohnt die Aufzaehlung; bei einem einzigen
|
||||
waere sie nur eine Verdopplung von "Übersicht". */}
|
||||
{dashboards.length > 1 && dashboards.map((d, index) => (
|
||||
<NavLink key={d.id} to={index === 0 ? "/" : `/d/${d.id}`} end
|
||||
<NavLink key={d.id} to={index === 0 ? "/home" : `/d/${d.id}`}
|
||||
className={({ isActive }) => (isActive ? "nav-link sub active" : "nav-link sub")}>
|
||||
<span className="label">{d.name}</span>
|
||||
</NavLink>
|
||||
@@ -206,7 +208,7 @@ function Protected({ children, adminOnly = false, wide = false }) {
|
||||
if (loading) return <div className="center muted" style={{ padding: 60 }}>Lädt…</div>;
|
||||
if (startFehler) return <StartFehler text={startFehler} erneut={erneutVersuchen} />;
|
||||
if (!user) return <Navigate to="/login" replace />;
|
||||
if (adminOnly && !isAdmin) return <Navigate to="/" replace />;
|
||||
if (adminOnly && !isAdmin) return <Navigate to="/home" replace />;
|
||||
return (
|
||||
<div className="app-shell">
|
||||
<Sidebar />
|
||||
@@ -215,8 +217,14 @@ function Protected({ children, adminOnly = false, wide = false }) {
|
||||
<div className={wide ? "content content--wide" : "content"}>
|
||||
{/* Je Seite eine eigene Grenze: stuerzt eine ab, bleibt wenigstens
|
||||
die Navigation bedienbar. `key` setzt sie beim Seitenwechsel
|
||||
zurueck, sonst haenge man nach einem Fehler dauerhaft fest. */}
|
||||
<ErrorBoundary key={pfad} bereich="Diese Seite">{children}</ErrorBoundary>
|
||||
zurueck, sonst haenge man nach einem Fehler dauerhaft fest.
|
||||
|
||||
Nur das ERSTE Pfadsegment: Sonst gilt /home/edit als andere Seite
|
||||
als /home, der ganze Baum wird neu eingehaengt und laedt beim
|
||||
Umschalten in den Bearbeitungsmodus alles noch einmal. */}
|
||||
<ErrorBoundary key={pfad.split("/")[1] || "home"} bereich="Diese Seite">
|
||||
{children}
|
||||
</ErrorBoundary>
|
||||
</div>
|
||||
</div>
|
||||
</div>
|
||||
@@ -230,10 +238,19 @@ export default function App() {
|
||||
<Routes>
|
||||
<Route
|
||||
path="/login"
|
||||
element={loading ? null : user ? <Navigate to="/" replace /> : <Login />}
|
||||
element={loading ? null : user ? <Navigate to="/home" replace /> : <Login />}
|
||||
/>
|
||||
<Route path="/" element={<Protected wide><Dashboard /></Protected>} />
|
||||
{/* Die Startseite hat einen Namen, damit in der Adresszeile steht, wo
|
||||
man ist; "/" leitet sichtbar dorthin um. Der Bearbeitungsmodus ist
|
||||
ein eigener Pfad statt eines Zustands im Speicher: So ueberlebt er
|
||||
ein Neuladen, laesst sich verschicken, und die Zurueck-Taste
|
||||
verlaesst ihn. Den Modus gibt die Prop vor, damit der Pfad nicht ein
|
||||
zweites Mal in der Komponente steht. */}
|
||||
<Route path="/" element={<Navigate to="/home" replace />} />
|
||||
<Route path="/home" element={<Protected wide><Dashboard /></Protected>} />
|
||||
<Route path="/home/edit" element={<Protected wide><Dashboard bearbeiten /></Protected>} />
|
||||
<Route path="/d/:id" element={<Protected wide><Dashboard /></Protected>} />
|
||||
<Route path="/d/:id/edit" element={<Protected wide><Dashboard bearbeiten /></Protected>} />
|
||||
<Route path="/checkin" element={<Protected><CheckIn /></Protected>} />
|
||||
<Route path="/checkout" element={<Protected><CheckOut /></Protected>} />
|
||||
<Route path="/lebensmittel" element={<Protected wide><Products fixedType="food" /></Protected>} />
|
||||
@@ -270,7 +287,7 @@ export default function App() {
|
||||
<Route path="/auth/callback" element={<OidcCallback />} />
|
||||
{/* Ebenfalls oeffentlich: hier ist man noch nicht angemeldet. */}
|
||||
<Route path="/sso-neu" element={<SsoNeu />} />
|
||||
<Route path="*" element={<Navigate to="/" replace />} />
|
||||
<Route path="*" element={<Navigate to="/home" replace />} />
|
||||
</Routes>
|
||||
);
|
||||
}
|
||||
|
||||
Reference in New Issue
Block a user