bug: align the Notes navbar with the full-width student workspace header
Les mainteneurs répondent en général sous 1 jour
Personne n'a encore pris cette issue.
Évaluation
- Difficulté
- 2/5
- Temps estimé
- 1-3 heures
- Accessibilité débutants
- 62/100
- Type d'issue
- Bug
- Clarté
- Plutôt claire
- Activité
- Active
- Stack technique
- nextjs, react, typescript
- Domaine
- frontend
Piste de recherche
Commence dans src/components/site-header.tsx, où la condition de route de l'espace de travail liste /account, /learn et /practice mais pas /dashboard, et où Notes reçoit max-w-7xl et un positionnement relative. Compare les classes de l'en-tête de Notes au traitement de Practice, puis vérifie Notes et les routes imbriquées dans un navigateur à 360px, 768px et 1440px. C'est terminé lorsque Notes et Practice partagent l'alignement du logo et du menu de compte ainsi que les dimensions de l'en-tête, qu'il n'y a aucun débordement horizontal et que les zones tactiles font au moins 44px.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
What happened?
The Notes section does not use the same full-width navbar alignment as the student workspace. In the owner's production screenshot of /dashboard, the logo and account menu are inset inside a centered container. In the supplied /practice reference, the logo sits near the left viewport edge and the account menu sits near the right edge.
Practice is the approved reference for the Notes navbar. Shared navigation should feel consistent across the application.
Steps to reproduce
- Open https://procoderji.com/dashboard on desktop.
- Observe the horizontal positions of the logo, main navigation and account menu.
- Open https://procoderji.com/practice at the same viewport width.
- Compare: the Notes header is constrained, while the Practice header uses the available width.
Expected behavior
Use the existing Practice navbar's full-width alignment, spacing, typography, white background, subtle border and scrolling behavior for Notes and its nested pages. Continue using one shared SiteHeader with the same logo, links/order and account menu; highlight the current section.
Code pointers
In src/components/site-header.tsx, the workspace route condition includes /account, /learn and /practice but excludes /dashboard. It selects w-full and sticky positioning for workspace routes, while Notes receives max-w-7xl and relative positioning. Review this route treatment before introducing any additional header component.
This observation comes from the current source and owner-supplied screenshots; desktop/mobile browser verification remains part of implementation.
Acceptance criteria
- At the same desktop viewport, Notes and Practice have matching logo/account-menu alignment and header dimensions.
- Nested Notes routes retain the same shared-header treatment and active Browse Notes link.
- Header scrolling behavior is consistent with the approved student-workspace reference.
- At 360px, 768px and 1440px, menus remain usable with no horizontal overflow and at least 44px tap targets.
- Preserve the existing Notes catalogue/reader functionality, current copy and yellow brand design.
- Keep Home's hero transparency behavior covered separately by #101, and signed-in display names by #102.
Planning
Backlog bug report. Record the Practice screenshot as the expected reference; implementation will be scheduled separately through an issue-linked contribution.
Production release at reporting: 9e2b9c00556de0387b09e937ffbefbd4eb9e0919.
- Langage dominant
- TypeScript
- Étoiles
- 1
- Forks
- 1
- Merge moyen
- 8 h 58 min
- PR mergées (30 j)
- 37
Préparer son environnement
- Fournit un Dockerfile ou un fichier Docker Compose
- Propose un modèle de pull request
- Lire le guide de contribution
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Autres issues de procoderji-com/procoderji
-
enhancement good first issue
Difficulté 2/5 1-3 heures Accessibilité débutants 88/100
procoderji-com/procoderji#36 ·
Les mainteneurs répondent en général sous 1 jour
-
enhancement good first issue
Difficulté 1/5 1-3 heures Accessibilité débutants 85/100
procoderji-com/procoderji#35 ·
Les mainteneurs répondent en général sous 1 jour
-
enhancement good first issue
Difficulté 1/5 1-3 heures Accessibilité débutants 79/100
procoderji-com/procoderji#34 ·
Les mainteneurs répondent en général sous 1 jour
-
enhancement good first issue
Difficulté 1/5 1-3 heures Accessibilité débutants 90/100
procoderji-com/procoderji#33 ·
Les mainteneurs répondent en général sous 1 jour
-
enhancement good first issue
Difficulté 1/5 1-3 heures Accessibilité débutants 82/100
procoderji-com/procoderji#32 ·
Les mainteneurs répondent en général sous 1 jour
Toutes les issues de procoderji-com/procoderji
Issues similaires
-
Difficulté 2/5 1-3 heures Accessibilité débutants 72/100
Les mainteneurs répondent en général sous 1 jour
-
bug platform-android
Difficulté 2/5 1-3 heures Accessibilité débutants 68/100
Termix-SSH/Mobile#146 · 1 commentaire ·
Les mainteneurs répondent en général sous 2 jours
-
bug
Difficulté 2/5 1-3 heures Accessibilité débutants 66/100
devswha/herdr-web-ui#3875 ·
-
Difficulté 1/5 Moins d'une heure Accessibilité débutants 88/100
-
area: runtime chore cli consistency dependencies developer-experience difficulty: easy discovery documentation enhancement good first issue help wanted hygiene json-output packaging priority: P3 tests version
Difficulté 2/5 1-3 heures Accessibilité débutants 78/100
Les mainteneurs répondent en général sous 1 jour