feat(builder-carto): réintégrer <app-header> commun (nav inter-apps) - #481
Merged
Conversation
La refonte carte plein écran (#472) avait retiré l'app-header partagé au profit d'une topbar carto minimaliste. On perdait ainsi la navigation vers les autres apps (Sources, Builder, Dashboard...). Solution : superposer les deux barres, header commun en haut avec la nav DSFR + les liens outils, topbar carto simplifiée en dessous avec les actions carto-spécifiques (Exécuter / Obtenir le code). Les liens redondants (logo RF, Guide, bouton Favoris) sont retirés de la topbar carto — le bouton « Sauvegarder en favori » reste accessible depuis la modale d'export. Le layout passe de position:absolute plein-viewport à flex column, avec une CSS variable --carto-header-h mesurée au runtime via ResizeObserver (le fr-header varie selon breakpoint). state.map.height inclut désormais cette variable dans son calc(). Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Contexte
La refonte carte plein écran (PR #472) avait volontairement retiré l'
app-headerpartagé au profit d'une topbar carto minimaliste — on perdait la navigation vers les autres apps du monorepo (Sources, Builder, Dashboard, Playground…), ce qui isolait le builder-carto du reste de l'écosystème.Approche (option B validée en session)
Superposer les deux barres :
<app-header>) en haut avec la nav DSFR + les liens outils (Guide, Specs, Favoris, Mon espace)Les liens redondants sont retirés de la topbar carto :
<app-header><app-header>btn-export-favorite)Layout
Passage de
position: absolute; height: 100dvh(plein-viewport) àbody { display: flex; flex-direction: column }avec le workspace enflex: 1.Une CSS variable
--carto-header-hest mesurée au runtime viaResizeObserversur<app-header>(lefr-headervarie selon breakpoint : mobile menu stack, tagline sur 2 lignes desktop…). Cette variable est consommée par :state.map.height:calc(100dvh - var(--carto-header-h, 96px) - 56px [- 208px])Les offsets internes du workspace (canvas
top:56px, panelstop:72px) restent inchangés puisqu'ils sont relatifs au workspace, pas au body.Fichiers touchés
apps/builder-carto/index.html— ajout du<script app-ui.esm.js>+<app-header>+ simplification de la topbar cartoapps/builder-carto/src/main.ts—observeHeaderHeight()(ResizeObserver → var CSS), retrait du bindingbtn-favorite, ajustement du calcstate.map.heightapps/builder-carto/src/styles/carto.css— body en flex column, workspace en flex:1, var--carto-header-hAucun changeset : rien dans
packages/core/srcoupackages/shared.Test plan
--carto-header-hs'actualise, la carte se réajusteRefs
🤖 Generated with Claude Code