🚀 Production Release: test → prod by EnzoBagnis · Pull Request #243 · EnzoBagnis/SAE

EnzoBagnis · 2026-03-11T20:50:53Z

🚀 Déploiement en Production

✅ Auto-generated tests passed on test

Cette PR a été automatiquement créée par le pipeline CI/CD après validation des tests automatiquement générés sur la branche test.

⚠️ Actions requises avant merge :

Review approfondie du code
Vérification des changements critiques
Validation par au moins 1 reviewer
Approbation finale pour production

📋 Checklist de déploiement :

Les tests unitaires auto-générés passent
Aucune régression détectée
Documentation à jour
Prêt pour la production

Créée automatiquement par GitHub Actions

fix: remove unused navigation menus from multiple files

🔄 Auto-sync: dev → test

Copilot

Pull request overview

This production release PR promotes the test branch to prod and includes UI template changes that remove the desktop header navigation (<nav class="nav-menu">) across multiple views.

Changes:

Removed <nav class="nav-menu"> blocks from user, resources, exercises, and admin view templates.
Standardized headers toward burger-based navigation markup in some pages (without updating the responsive CSS in this PR).

Reviewed changes

Copilot reviewed 11 out of 11 changed files in this pull request and generated 6 comments.

Show a summary per file

File	Description
App/View/user/resources_list.php	Removes an empty `nav-menu` container from the header.
App/View/user/resource_details.php	Removes an empty `nav-menu` container from the header.
App/View/user/mentions-legales.php	Removes an empty `nav-menu` container from the header.
App/View/user/ia.php	Removes desktop `nav-menu` links (Ressources/IA) from the header.
App/View/user/dashboard.php	Removes desktop `nav-menu` links (Ressources/IA) from the header.
App/View/resources/list.php	Removes desktop `nav-menu` from the header.
App/View/resources/details.php	Removes desktop `nav-menu` links (Ressources/Exercices/IA) from the header.
App/View/resources/create.php	Removes desktop `nav-menu` from the header.
App/View/exercises/show.php	Removes desktop `nav-menu` links (Ressources/Exercices) from the header.
App/View/exercises/list.php	Removes desktop `nav-menu` links (Ressources/Exercices) from the header.
App/View/admin/admin-dashboard.php	Removes the admin `nav-menu` link from the header.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copilot · 2026-03-11T20:54:35Z

    <button class="burger-menu" id="burgerBtn" onclick="toggleBurgerMenu()" aria-label="Menu">
        <span></span><span></span><span></span>
    </button>
-    <nav class="nav-menu">
-        <a href="<?= BASE_URL ?>/resources">Ressources</a>
-        <a href="<?= BASE_URL ?>/ia" class="active">IA</a>
-    </nav>
    <div class="header-right">
        <div class="user-profile">


La suppression du
enlève la navigation visible sur desktop. Dans public/css/dashboard.css, .burger-menu est masqué par défaut (display:none) et n’apparaît qu’en <=768px; sans nav-menu, il n’y a donc plus d’accès direct à « Ressources/IA » sur grand écran. Solution : réintroduire un nav-menu desktop, ou modifier le CSS pour afficher le burger (et son contenu) à toutes les tailles d’écran si c’est l’intention.

Copilot · 2026-03-11T20:54:36Z

    <button class="burger-menu" id="burgerBtn" onclick="toggleBurgerMenu()" aria-label="Menu">
        <span></span><span></span><span></span>
    </button>
-    <nav class="nav-menu">
-        <a href="<?= BASE_URL ?>/resources" class="active">Ressources</a>
-        <a href="<?= BASE_URL ?>/ia">IA</a>
-    </nav>
    <div class="header-right">
        <button onclick="openImportModal(<?= $current_resource_id ?>)" class="btn-import-trigger">


Ici, le
a été retiré alors que le bouton burger est caché sur desktop (dashboard.css: .burger-menu { display:none } hors media query). Résultat : plus de navigation visible (Ressources/IA) sur écran large. À corriger en rétablissant la nav desktop ou en rendant le burger toujours visible (avec les styles associés).

Copilot · 2026-03-11T20:54:36Z

    <button class="burger-menu" id="burgerBtn" onclick="toggleBurgerMenu()" aria-label="Menu">
        <span></span><span></span><span></span>
    </button>
-    <nav class="nav-menu">
-        <a href="<?= BASE_URL ?>/resources" class="active">Ressources</a>
-        <a href="<?= BASE_URL ?>/exercises">Exercices</a>
-        <a href="<?= BASE_URL ?>/ia">IA</a>
-    </nav>
    <div class="header-right">
        <button onclick="openImportModal(<?= (int)$resource->getResourceId() ?>)" class="btn-import-trigger">


La nav desktop (
) a été supprimée mais le bouton burger reste masqué sur desktop (dashboard.css). Cela supprime l’accès visible aux sections (Ressources/Exercices/IA) sur grand écran. Soit rétablir le nav-menu, soit ajuster les styles pour afficher le burger sur desktop (et vérifier que le burger-nav contient bien tous les liens nécessaires, dont IA si attendu).

Copilot · 2026-03-11T20:54:36Z

@@ -31,10 +31,6 @@
        <span></span><span></span><span></span>
    </button>



La suppression du
retire la navigation visible sur desktop, car le bouton burger est caché hors mobile (dashboard.css). Sans nav-menu, il n’y a plus de lien direct entre « Ressources » et « Exercices » sur grand écran. Solution : remettre le nav-menu pour desktop ou rendre le burger disponible sur toutes les largeurs d’écran.

Suggested change

<nav class="nav-menu">

<ul>

<li><a href="<?= BASE_URL ?>/index.php?action=resources_list">Ressources</a></li>

<li><a href="<?= BASE_URL ?>/index.php?action=exercises" class="active">Exercices</a></li>

</ul>

</nav>

Copilot · 2026-03-11T20:54:37Z

@@ -32,10 +32,6 @@
        <span></span><span></span><span></span>
    </button>



Même problème ici : en supprimant
, on perd la navigation desktop puisque .burger-menu est masqué par défaut (dashboard.css) et n’apparaît qu’en mobile. Cela rend la page « Exercices » difficilement navigable sur grand écran. Réintroduire le nav-menu ou rendre le burger visible sur desktop.

Suggested change

<nav class="nav-menu">

<ul>

<li><a href="<?= BASE_URL ?>/index.php?action=resources_list">Ressources</a></li>

<li><a href="<?= BASE_URL ?>/index.php?action=exercises" class="active">Exercices</a></li>

<li><a href="#" onclick="confirmLogout()">Déconnexion</a></li>

</ul>

</nav>

Copilot · 2026-03-11T20:54:37Z

@@ -14,9 +14,6 @@
    <div class="logo">
        <h1>StudTraj</h1>
    </div>


La suppression du
enlève le seul lien de navigation dans le header. Si l’admin n’a qu’une page, ça peut être acceptable, mais sinon la navigation desktop disparaît (admin.css stylise explicitement .nav-menu). Vérifier l’intention : soit conserver ce lien, soit mettre en place une navigation alternative cohérente avec le reste du layout admin.

Suggested change

</div>

</div>

<nav class="nav-menu">

<a href="<?= BASE_URL ?>/admin">Tableau de bord</a>

</nav>

EnzoBagnis added 3 commits March 11, 2026 21:48

fix: remove unused navigation menus from multiple files

c4c0d82

Merge pull request #241 from EnzoBagnis/Merge-IA-sur-graphes

f7b7b5e

fix: remove unused navigation menus from multiple files

Merge pull request #242 from EnzoBagnis/dev

0ea978d

🔄 Auto-sync: dev → test

Copilot AI review requested due to automatic review settings March 11, 2026 20:50

Copilot started reviewing on behalf of EnzoBagnis March 11, 2026 20:51 View session

MARCUS-William-2326073aa approved these changes Mar 11, 2026

View reviewed changes

PibouleauJB approved these changes Mar 11, 2026

View reviewed changes

EnzoBagnis requested review from STEFANOVITCH-Ilann-24019037 and koliaih March 11, 2026 20:52

EnzoBagnis merged commit ba10078 into prod Mar 11, 2026
16 checks passed

Copilot AI reviewed Mar 11, 2026

View reviewed changes

Navigation Menu

🚀 Production Release: test → prod - #243

🚀 Production Release: test → prod#243
EnzoBagnis merged 3 commits into
prodfrom
test

EnzoBagnis commented Mar 11, 2026

Uh oh!

Uh oh!

Copilot AI left a comment

Uh oh!

Copilot AI Mar 11, 2026

Uh oh!

Copilot AI Mar 11, 2026

Uh oh!

Copilot AI Mar 11, 2026

Uh oh!

Copilot AI Mar 11, 2026

Uh oh!

Copilot AI Mar 11, 2026

Uh oh!

Copilot AI Mar 11, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

		@@ -31,10 +31,6 @@
		<span></span><span></span><span></span>
		</button>

+    <nav class="nav-menu">
+        <ul>
+            <li><a href="<?= BASE_URL ?>/index.php?action=resources_list">Ressources</a></li>
+            <li><a href="<?= BASE_URL ?>/index.php?action=exercises" class="active">Exercices</a></li>
+        </ul>
+    </nav>

		@@ -32,10 +32,6 @@
		<span></span><span></span><span></span>
		</button>

Conversation

EnzoBagnis commented Mar 11, 2026

🚀 Déploiement en Production

⚠️ Actions requises avant merge :

📋 Checklist de déploiement :

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Reviewed changes

Uh oh!

Copilot AI Mar 11, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Mar 11, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Mar 11, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Mar 11, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Mar 11, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Mar 11, 2026

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants