HNNW/core
Watch
28
2
Fork
You've already forked core
 0
Code Issues Pull requests 8 Releases Activity Actions

team-2/develop #53

Open
wijayant-jagroep wants to merge 66 commits from team-2/develop into develop
pull from: team-2/develop
merge into: HNNW:develop
Conversation 2 Commits 66 Files changed 27 +1830 -659
wijayant-jagroep commented 2026-10-12 08:40:14 +00:00
Member
Copy link
No description provided. 
Reviewed-on: #17
This reverts commit 1c9809cdcc, reversing
changes made to 5efbc905c8.
Reviewed-on: #29
Reviewed-on: #30
Remove the duplicate homepage navigation, generate shared navigation URLs from Symfony routes, replace the handshake emoji with a UX icon, and clean up template whitespace.
Explain the homepage wrapper, scoped styles, and globally named keyframes.
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Refine the homepage value proposition, translate shared navigation into Dutch, and improve responsive menu behavior.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Restore the previous shared header labels, menu script, and core header styles.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Replace the manually positioned share icon with a centered SVG in the feature tile.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Tighten homepage copy and section spacing, refine feature icon alignment, and improve mobile typography and CTA layout.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Reviewed-on: #43
Reviewed-on: #32
Reviewed-on: #47
Reviewed-on: #48
Reviewed-on: #34
rileydeman requested changes 2026-10-12 12:47:18 +00:00
rileydeman left a comment
Copy link

Mooi werk aan de nieuwe home- en aboutpagina. De teksten zijn helder en in het Nederlands. De CSS staat netjes onder .home-page/.about-page, dus andere pagina's merken er niets van. Je gebruikt overal asset(), path('auth.register') en ux_icon() in plaats van harde paden, en dat is precies goed.

Dit moet nog aangepast worden:

Moet opgelost worden

  1. De PR verwijdert werk van develop dat niet weg mag. Commit f466c10 ("Revert "Merge branch 'team-2/develop' into …home-page"") draaide een merge terug waarin eigenlijk develop zat (PR #5, Events). Daardoor verwijdert deze PR bij het mergen: src/Entity/Common/Event.php, migrations/Version20260921111743.php, de event-relaties in Account.php en Organisation.php, DashboardEventsController, PanelEventsController, de routes main.events/main.event in MainController en alle templates/pages/*/events/*. Ook de composer-fix wordt teruggedraaid (doctrine/orm vastgezet op 3.6.7, plus composer.lock). Wie die migratie al gedraaid heeft, heeft dan een database die niet meer bij de code past. Fix: voer git revert f466c10 uit op team-2/develop. Dat gaat zonder conflicten. Daarna laat de diff alleen nog jullie eigen bestanden zien.
  2. src/Entity/Project/WorkPackage.php: door dezelfde revert heet de property weer $packageTasks, terwijl de constructor, getWorkPackageTasks(), addWorkPackageTask() en removeWorkPackageTask() nog $this->workPackageTasks gebruiken. Zodra je de taken van een work package opvraagt, krijg je een "undefined property"-fout. Dit was op develop al opgelost (commit 040225e). Punt 1 lost dit ook op.
  3. .env bevat nu een echte APP_SECRET en postgres:postgres als databasegegevens. .env staat in git en de repo is publiek, dus dit secret is nu openbaar. Fix: zet .env terug naar de versie van develop (git checkout origin/develop -- .env) en zet je eigen waarden in .env.local, dat niet in git komt. Laat ook de commentaarregels (Mercure-uitleg) staan.
  4. about.html.twig: de knop "Naar de open bèta" linkt naar index.html#aanmelden. Dat bestand bestaat niet in Symfony, dus de link geeft een 404. Gebruik {{ path('auth.register') }}, net als op de homepage.

Verbeterpunten
5. about.html.twig heeft een eigen <div class="nav"> met logo en links. layouts/main.base.html.twig laadt de gedeelde header al, dus bezoekers zien nu twee navigaties. Haal de eigen nav weg. Mis je links in de header? Pas dan de gedeelde _header.html.twig aan, en overleg dat met de andere teams.
6. about.html.twig zet een <main> binnen de <main> van de base layout. Dat is ongeldige HTML. Gebruik een <div> of laat hem weg. Haal ook het lege <!-- FOOTER -->-blok en de vele lege regels weg.
7. about.html.twig: seoKeywords en seoDescription zijn leeg en de titel is "About". Vul ze in zoals op de homepage, en maak er "Over ons" van.
8. index.css en about.css hebben elk een eigen kleur en lettertype (#4da3ff/Arial tegenover #4b9cf5/Inter). Ook .button, .eyebrow en de hero-visual staan in beide bestanden. Daardoor zien de pagina's er net anders uit. Maak één set variabelen en gedeelde classes, het liefst op basis van de variabelen in assets/shared/css/style.css.
9. index.html.twig: de hero heeft id="about" en de features-sectie id="projects", maar het zijn geen about- of projectensecties. Geef ze namen die bij de inhoud passen (bijv. #inhoud, #voordelen) en laat de skip-link daarnaar wijzen.
10. Zet een korte beschrijving in de PR: wat er verandert en hoe je het test. Met 66 commits en een paar merges heen en weer helpt dat de reviewer echt.

Let op: PR #33 (team 5, website-styling) heeft team-2/develop als base en herschrijft ook about.html.twig, index.html.twig, about.css, index.css en de gedeelde _header.html.twig. Net als deze PR past #33 ook de DATABASE_URL in .env aan. Stem de merge-volgorde af, anders krijg je grote conflicten. Ook #15 (team 6) wijzigt .env.

Mooi werk aan de nieuwe home- en aboutpagina. De teksten zijn helder en in het Nederlands. De CSS staat netjes onder `.home-page`/`.about-page`, dus andere pagina's merken er niets van. Je gebruikt overal `asset()`, `path('auth.register')` en `ux_icon()` in plaats van harde paden, en dat is precies goed. Dit moet nog aangepast worden: **Moet opgelost worden** 1. De PR verwijdert werk van develop dat niet weg mag. Commit `f466c10` ("Revert "Merge branch 'team-2/develop' into …home-page"") draaide een merge terug waarin eigenlijk `develop` zat (PR #5, Events). Daardoor verwijdert deze PR bij het mergen: `src/Entity/Common/Event.php`, `migrations/Version20260921111743.php`, de event-relaties in `Account.php` en `Organisation.php`, `DashboardEventsController`, `PanelEventsController`, de routes `main.events`/`main.event` in `MainController` en alle `templates/pages/*/events/*`. Ook de composer-fix wordt teruggedraaid (`doctrine/orm` vastgezet op `3.6.7`, plus `composer.lock`). Wie die migratie al gedraaid heeft, heeft dan een database die niet meer bij de code past. Fix: voer `git revert f466c10` uit op `team-2/develop`. Dat gaat zonder conflicten. Daarna laat de diff alleen nog jullie eigen bestanden zien. 2. `src/Entity/Project/WorkPackage.php`: door dezelfde revert heet de property weer `$packageTasks`, terwijl de constructor, `getWorkPackageTasks()`, `addWorkPackageTask()` en `removeWorkPackageTask()` nog `$this->workPackageTasks` gebruiken. Zodra je de taken van een work package opvraagt, krijg je een "undefined property"-fout. Dit was op develop al opgelost (commit `040225e`). Punt 1 lost dit ook op. 3. `.env` bevat nu een echte `APP_SECRET` en `postgres:postgres` als databasegegevens. `.env` staat in git en de repo is publiek, dus dit secret is nu openbaar. Fix: zet `.env` terug naar de versie van develop (`git checkout origin/develop -- .env`) en zet je eigen waarden in `.env.local`, dat niet in git komt. Laat ook de commentaarregels (Mercure-uitleg) staan. 4. `about.html.twig`: de knop "Naar de open bèta" linkt naar `index.html#aanmelden`. Dat bestand bestaat niet in Symfony, dus de link geeft een 404. Gebruik `{{ path('auth.register') }}`, net als op de homepage. **Verbeterpunten** 5. `about.html.twig` heeft een eigen `<div class="nav">` met logo en links. `layouts/main.base.html.twig` laadt de gedeelde header al, dus bezoekers zien nu twee navigaties. Haal de eigen nav weg. Mis je links in de header? Pas dan de gedeelde `_header.html.twig` aan, en overleg dat met de andere teams. 6. `about.html.twig` zet een `<main>` binnen de `<main>` van de base layout. Dat is ongeldige HTML. Gebruik een `<div>` of laat hem weg. Haal ook het lege `<!-- FOOTER -->`-blok en de vele lege regels weg. 7. `about.html.twig`: `seoKeywords` en `seoDescription` zijn leeg en de titel is "About". Vul ze in zoals op de homepage, en maak er "Over ons" van. 8. `index.css` en `about.css` hebben elk een eigen kleur en lettertype (`#4da3ff`/Arial tegenover `#4b9cf5`/Inter). Ook `.button`, `.eyebrow` en de hero-visual staan in beide bestanden. Daardoor zien de pagina's er net anders uit. Maak één set variabelen en gedeelde classes, het liefst op basis van de variabelen in `assets/shared/css/style.css`. 9. `index.html.twig`: de hero heeft `id="about"` en de features-sectie `id="projects"`, maar het zijn geen about- of projectensecties. Geef ze namen die bij de inhoud passen (bijv. `#inhoud`, `#voordelen`) en laat de skip-link daarnaar wijzen. 10. Zet een korte beschrijving in de PR: wat er verandert en hoe je het test. Met 66 commits en een paar merges heen en weer helpt dat de reviewer echt. Let op: PR #33 (team 5, `website-styling`) heeft `team-2/develop` als base en herschrijft ook `about.html.twig`, `index.html.twig`, `about.css`, `index.css` en de gedeelde `_header.html.twig`. Net als deze PR past #33 ook de `DATABASE_URL` in `.env` aan. Stem de merge-volgorde af, anders krijg je grote conflicten. Ook #15 (team 6) wijzigt `.env`.
patrick requested changes 2026-10-13 12:21:18 +00:00
patrick left a comment
Copy link

Voor de home en about pagina zijn nog de volgende requirments van de stakeholders erbij gekomen:

  1. Op de homepagina bied ‘Bekijk projecten’ en ‘Maak een account’ of ‘Mijn account’ passend bij de sessie.
  2. Op de homepainga Toon maximaal zes gepubliceerde, voor aanmelding open projecten met titel, organisatie en samenvatting; bij geen projecten een passende melding.
  3. Op de about pagina link naar projecten.
Voor de home en about pagina zijn nog de volgende requirments van de stakeholders erbij gekomen: 1. Op de homepagina bied ‘Bekijk projecten’ en ‘Maak een account’ of ‘Mijn account’ passend bij de sessie. 2. Op de homepainga Toon maximaal zes gepubliceerde, voor aanmelding open projecten met titel, organisatie en samenvatting; bij geen projecten een passende melding. 3. Op de about pagina link naar projecten.
This pull request doesn't have enough approvals yet. 0 of 2 approvals granted.
You are not authorized to merge this pull request.
View command line instructions

Checkout

From your project repository, check out a new branch and test the changes.
git fetch -u origin team-2/develop:team-2/develop
git switch team-2/develop
Sign in to join this conversation.
Reviewers
No reviewers
rileydeman
patrick
Labels
Clear labels
No items
No labels
Milestone
Clear milestone
No items
No milestone
Assignees
Clear assignees
No assignees
3 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set

Reference
HNNW/core!53
No description provided.