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

test(components-react): add visual baseline section #346

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

remypar5
Copy link
Contributor

@remypar5 remypar5 commented Dec 6, 2024

Contents

Maak een baseline voor de visuele regressietests voor het LuxSection React component.

Checklist

  • New features/components and bugfixes are covered by tests
  • Changesets are created
  • Definition of Done is checked

@remypar5 remypar5 self-assigned this Dec 6, 2024
Copy link

vercel bot commented Dec 6, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
lux ✅ Ready (Inspect) Visit Preview 💬 Add feedback Dec 20, 2024 1:20pm

@remypar5 remypar5 removed the request for review from MMeijerink December 6, 2024 13:12
Copy link
Contributor

@AlineNap AlineNap left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is het mogelijk om de canvas kleur toe te voegen aan het voorbeeld in de playground van Storybook? Dan zie je naast het kleurverschil ook de border-radius.

Verder top!

@remypar5
Copy link
Contributor Author

remypar5 commented Dec 19, 2024

Is het mogelijk om de canvas kleur toe te voegen aan het voorbeeld in de playground van Storybook? Dan zie je naast het kleurverschil ook de border-radius.

@AlineNap Zeker. Je kan verschillende achtergrondkleuren of zelfs een -patroon kiezen. Dat doe je met het 2e knopje links van de theme switcher. Voor jou zelfs een paarse kleur beschikbaar gemaakt

@AlineNap
Copy link
Contributor

Ik zie niks veranderen als ik paars selecteer.

Wat ik bedoel is dat ik graag in de playground de token lux.color.canvas toegepast zie als achtergrond kleur met daarop de section. Dan is het duidelijk dat de section een border radius heeft en waar die stopt qua padding.

Dus dan zou de paarse omlijning (zie screenshots) de section zijn en de achtergrond is dan de canvas kleur. Als ik dan switch naar darkmode zal de canvas kleur zich ook aanpassen.

image

Screenshot 2024-12-19 at 16 58 35

@remypar5
Copy link
Contributor Author

Wat ik bedoel is dat ik graag in de playground de token lux.color.canvas toegepast zie als achtergrond kleur met daarop de section. Dan is het duidelijk dat de section een border radius heeft en waar die stopt qua padding.

@AlineNap Ik snap m en heb canvas als optie toegevoegd om als achtergrond in te stellen. Deze ook voor de Section als standaard ingesteld.

Ik begrijp alleen niet waarom je geen verandering ziet als je boundaries als achtergrond kiest. Er zou dus een paarse rand om de section heen moeten komen te staan.

@remypar5 remypar5 requested a review from AlineNap December 20, 2024 13:25
@remypar5 remypar5 enabled auto-merge (squash) December 20, 2024 14:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants