chore(deps): bump vendored front-end libs, pin Actions op SHA + GLightbox-demosectie - #16
Merged
Merged
Conversation
…p SHA Vendor libs (webroot/js, webroot/css), opgehaald van de npm-registry: bootstrap 5.3.5 (js) / 5.3.3 (css) -> 5.3.8 (waren uit sync) swiper 11.2.6 (js) / 11.2.4 (css) -> 14.1.0 (waren uit sync) glightbox 3.3.0 -> 3.3.1 bootstrap-icons 1.11.3 -> 1.13.1 remixicon 4.6.0 -> 4.9.1 boxicons -> 2.1.4 purecounter -> 1.5.0 jquery 3.7.1 -> 4.0.0 (bestand hernoemd) isotope 3.0.6 -> ongewijzigd, is al de laatste release (2018) waypoints 4.0.1 -> ongewijzigd, is al de laatste release (2016) popper 2.11.8 -> ongewijzigd, is al de laatste release De font-paden in de icon-CSS zijn weer platgeslagen (./fonts/ en ../fonts/ weg), omdat de fonts in deze repo naast de CSS in webroot/css staan. Verificatie met Chromium: index.html en 404.html laden zonder JS-fouten en zonder enkele lokale 404; bootstrap meldt zich als 5.3.8; de woff2's van bootstrap-icons, remixicon en boxicons komen met hun nieuwe cache-hashes binnen en de icons renderen. Swiper/GLightbox/Isotope/PureCounter/Waypoints laden wel, maar er staat geen bijbehorende markup op de pagina's, dus de major bumps raken hier niets aan runtime-gedrag. Workflow: docker/login-action, docker/metadata-action en docker/build-push-action stonden op een verplaatsbare tag. Nu op de commit-SHA gepind met de versie in een comment, wat CodeQL-alerts #2, #3 en #4 (actions/unpinned-tag) afsluit. Renovate blijft ze via die comment bijwerken. Let op: dit sluit alerts #5 en #6 (glightbox, missing regexp anchor) NIET. Die twee wijzen naar /vimeo\.com\/([0-9]*)/ op r.953 en r.1103, en glightbox 3.3.1 bevat exact diezelfde regel - hij schuift alleen op naar r.945 en 1095. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01KRc1UN8fR9YKnardtGxRCE
GLightbox en Isotope werden wel ingeladen, maar er stond geen enkel element op de pagina dat ze aanriep. Deze portfolio-sectie is een placeholder die laat zien hoe en waar je ze aanhaakt. De sectie gebruikt de markup waar de theme-CSS al op gebouwd is (.portfolio-wrap, .portfolio-info, .portfolio-links, #portfolio-flters), dus er was geen nieuwe opmaak voor nodig. In de HTML staat een comment met waar de init vandaan komt (webroot/js/main.js) en wat href, data-gallery, data-glightbox en data-type doen, plus voorbeelden voor video en voor de tweede instance (.portfolio-details-lightbox, 90% / 90vh). Drie tegels met de afbeeldingen die al in de repo staan, in een gedeelde gallery zodat je ertussen kunt bladeren. Nav-item "Portfolio" toegevoegd, zodat de sectie via de bestaande section-show-flow bereikbaar is. In style.css een regel die de tegels op 4/3 met object-fit: cover zet. De meegeleverde afbeeldingen hebben verschillende verhoudingen; zonder die regel staat het raster ongelijk zodra je er eigen beeld in hangt. Geverifieerd in Chromium: 3 lightbox-triggers gevonden, Isotope legt het raster op, klikken opent de lightbox met 3 slides, de afbeelding laadt echt (naturalWidth > 0), het onderschrift rendert, de next-pijl gaat door naar de volgende afbeelding, sluiten werkt, en het filter "People" houdt precies één tegel over. Geen JS-fouten, geen enkele lokale 404. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01KRc1UN8fR9YKnardtGxRCE
Stensel8
changed the title
DevTools meldde twee keer "An element doesn't have an autocomplete attribute": de browser herkent deze velden aan hun id/name als autofill- velden, maar zonder autocomplete-attribuut kan hij ze niet betrouwbaar invullen. #name -> autocomplete="name" #email -> autocomplete="email" Subject en message krijgen er bewust geen: dat zijn geen velden die autofill herkent, en DevTools markeerde ze dan ook niet. Geverifieerd via het CDP Audits-domein (dat is wat het Issues-paneel voedt): op de pagina komt nu geen enkele issue meer binnen. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01KRc1UN8fR9YKnardtGxRCE
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
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.
Sluit 3 van de 5 openstaande CodeQL-alerts, bumpt alle vendored libs, geeft GLightbox een demo-plek in de template en ruimt twee DevTools-meldingen op het contactformulier op. De overige 2 alerts kunnen niet in code opgelost worden — zie onderaan.
Vendor libs gebumpt
Bootstrap en Swiper stonden met JS en CSS op verschillende versies; die lopen nu weer gelijk.