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

Ozon Content: Toegevoegde teksten zorgen ervoor dat anchors niet meer identificeerbaar zijn #2582

Closed
tfrijsewijk opened this issue Mar 14, 2024 · 6 comments
Assignees
Labels
♿ accessibility 🐛 bug Something isn't working
Milestone

Comments

@tfrijsewijk
Copy link
Member

tfrijsewijk commented Mar 14, 2024

In de renvooiweergave krijg nieuwe tekst een underline. Als er in deze tekst een anchor zit, is deze niet meer te herkennen:

Screenshot 2024-03-14 155538

  • In bovenstaand screenshot is de anchor gefocussed. Het zwarte kader is de focus stijling.
  • Check of dit ook geldt voor <NieuweTekst> en/of alleen voor wijzigactie="voegtoe".

Bij verwijderde teksten zijn anchors wel te identificeren:

Screenshot 2024-03-14 155624

@DjangoBlok
Copy link

@tfrijsewijk Bij mijn weten hebben we nooit gespecificeerd dat toegevoegde tekst onderstreept moet worden. Ook precies om deze reden.
Voor het onderscheid tussen rood en groen hebben we een schaduw toegevoegd aan de groene vlakjes.

Wat mij betreft is de oplossing dat we de onderstreping er vanaf halen.

@tfrijsewijk
Copy link
Member Author

@tfrijsewijk Bij mijn weten hebben we nooit gespecificeerd dat toegevoegde tekst onderstreept moet worden. Ook precies om deze reden. Voor het onderscheid tussen rood en groen hebben we een schaduw toegevoegd aan de groene vlakjes.

Ik denk dat je gelijk hebt. Ik kom niets tegen. Het is in release 59.0.0 met #2240 geintroduceerd 🤷

Wat mij betreft is de oplossing dat we de onderstreping er vanaf halen.

Prima!

@tfrijsewijk tfrijsewijk assigned iterox and unassigned mduijsens Mar 14, 2024
@iterox
Copy link
Contributor

iterox commented Apr 16, 2024

Het opmerkelijke is dat ik dit niet in de master storybook van de dso-toolkit kan vinden.
@DjangoBlok Heb jij een reproductie pad of document waarin zich dit in de viewer voordoet?

@DjangoBlok
Copy link

Ik kan het ook niet reproduceren op de productieomgeving.

@tfrijsewijk
Copy link
Member Author

We sluiten 'm, mocht het zich weer voordoen kunnen we dit issue er weer bij pakken.

tfrijsewijk added a commit that referenced this issue Apr 24, 2024
…rgen ervoor dat anchors niet meer identificeerbaar zijn
@tfrijsewijk tfrijsewijk assigned tfrijsewijk and unassigned iterox Apr 24, 2024
@tfrijsewijk tfrijsewijk reopened this Apr 24, 2024
@tfrijsewijk
Copy link
Member Author

tfrijsewijk commented Apr 24, 2024

Gevonden & gefixed 085c6f1

@tfrijsewijk tfrijsewijk added this to the 🪜 62.20.0 milestone Apr 24, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
♿ accessibility 🐛 bug Something isn't working
Projects
Status: ✅ Done
Development

No branches or pull requests

4 participants