CountNick
commented
3 months ago Ha, fijn dat je hiermee bezig bent geweest. Ik heb nog wel wat vragen / opmerkingen, zoals je ziet in de comments. Je begeleidend schrijven bij de PR was wel wat summier. Dus heb zelf een Lighthouse check uitgevoerd bij een willekeurig project (Melkweg) om te zien welke issues je nu weg aan het werken bent.
Ik denk dat nog niet alles klopt. Is er een manier waarop jij dit in de browser test?
Lighthouse kan soms wat lastig te interpreteren zijn. Lighthouse maakt gebruik van de "axe-core". Er is ook een eigen Axe Devtools, die ik wel eens gebruik. Die geeft soms net wat meer uitleg. (Chrome extension)
Daarnaast geeft Lighthouse ook nog wat "best practices". Het lijkt me goed om die ook mee te nemen. In dit geval gaat het vooral om een CSS aanpassing, dus die is relatief eenvoudig.
Best practices zijn in dit geval non-normatief, wat zoveel betekent als dat ze niet verplicht zijn volgens de guidelines. Maar wel goed om even mee te pakken indien mogelijk.
Ik test dit nu door lokaal een npm link op te zetten voor de package. Zo kan ik lokaal changes maken en snel weer een lighthouse test draaien. Excuus, ik ben inderdaad vergeten de exacte errors te beschrijven. Die styling op het input element moet binnen het project worden gedaan waar de popup is toegevoegd, dat kan helaas niet binnen deze PR
Fixes the aria issues that were detected with lighthouse