solid-design-system / solid

Monorepo for Union Investment's Solid Design System.
https://solid-design-system.fe.union-investment.de/docs/
Other
21 stars 4 forks source link

docs: 📚 Screenshot test diff being triggered by JS in setCustomValidity #1394

Closed Vahid1919 closed 4 weeks ago

Vahid1919 commented 2 months ago

Description

Our custom JS in setCustomValidty triggers unnecessary screenshot comparisons for design approval. This creates confusion and additional effort to find relevant changes in a PR. Image

Eg: Here only 1 component was changed (map-marker), yet these are the screenshot diffs generated. Image

SetCustomValidty stories should be updated in order to avoid this. (Perhaps they could be removed from the screenshots as well, but that should be discussed with the team)

Where to put the documentation

This affects:

DoR

DoD