Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Korrigere noen HTML syntax feil #581

Open
1 of 3 tasks
steinho opened this issue Mar 28, 2022 · 6 comments
Open
1 of 3 tasks

Korrigere noen HTML syntax feil #581

steinho opened this issue Mar 28, 2022 · 6 comments
Assignees

Comments

@steinho
Copy link
Member

steinho commented Mar 28, 2022

https://validator.w3.org/nu/?doc=https%3A%2F%2Fartsdatabanken.no%2Flister%2Frodlisteforarter%2F2021%2F15090

  • Det er noen tag'er som ikke er avsluttet... bl.a. manger på index. siden, Men også flere plasser
  • bruk av paragraph inni span - burde kanskje vært div i stedet for span.
  • target="_new" på <anchor ..... ugyldig verdi..... skal sikker være _blank

Fin sjekkliste på w3.org testen .....
Sikker flere litt lavthengende problem å rette....

@matssa
Copy link
Contributor

matssa commented Apr 1, 2022

target="_new" ser ut til å komme fra koden vi henter inn fra hjemmesida, relatert til menyene i headeren.
Ellers ser det ut som at de p-elementene kan komme av teksten som ligger lagret i vurderingen. Skal se litt mer på dette for å komme til bunns i det.

@steinho
Copy link
Member Author

steinho commented Apr 2, 2022

target="_new" ser ut til å komme fra koden vi henter inn fra hjemmesida, relatert til menyene i headeren. Ellers ser det ut som at de p-elementene kan komme av teksten som ligger lagret i vurderingen. Skal se litt mer på dette for å komme til bunns i det.

Mener jeg fant dem i noen json filer - som brukes bl.a. for naturtyper - og som lå i prosjektet

@matssa
Copy link
Contributor

matssa commented Apr 8, 2022

Fant de target="_new" med et litt grundigere søk. De sto som target=\"_new\", så søket mitt ga ikke treff første gang.
Fikset opp i det samt fjernet en del duplikate paragrafer. Kanskje @helemork kan se på de paragrafene inni span og slike, da hun har bedre kontroll på markup-en inni vurderingssiden? Jeg kjører den sjekken på søkesiden også og ser litt.

@matssa
Copy link
Contributor

matssa commented Apr 8, 2022

Jobber forresten i branchen 'fix-html-errors' https://github.com/Artsdatabanken/assessments-frontend/tree/fix-html-errors

@helemork
Copy link
Member

@matssa Det er veldig mange endringer i den greina, og såvidt jeg husker er det gode grunner for at ting var som de var på noe av det. Spesielt den der

greia gikk på enkelte spesifikke feil som dukka opp. Ved å endre den er det noen som vi få paragrafene sine helt feil. Men det er veldig mye her, og jeg husker ikke nok tilbake til å bare gå over og lese og si det er greit. F.eks å bare fjerne avsnitt midt i kriterietolkningen - er det ok? vet ikke.

Kunne vi enten gått igjennom sammen, eller tatt flere pr'er på hver type ting? for det er mye som må testes på hver av dem.

@matssa
Copy link
Contributor

matssa commented Apr 19, 2022

@helemork vi kan godt ta en runde på fredag for min del.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants