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

[Form Field] Text Input #242

Open
8 tasks
rwittenberns opened this issue Sep 24, 2024 · 2 comments
Open
8 tasks

[Form Field] Text Input #242

rwittenberns opened this issue Sep 24, 2024 · 2 comments
Assignees
Labels
new component Adding a new component to the library react Issue involving a ReactJS component

Comments

@rwittenberns
Copy link
Contributor

rwittenberns commented Sep 24, 2024

Omschrijving

Image

Aanpak

We pakken utrecht textbox

Definition of done

Component

  • Component is beschikbaar in de index
  • Design tokens worden gebruikt

Storybook

  • Playground story
  • States en properties stories
  • Korte beschrijving van het component
    • Wat doet het
    • Basis van het component (bijv Utrecht)
    • Aanpassingen en/of uitbreidingen
    • Relatie met andere componenten (bijv. Form field label en Form field)

Tests

  • Het component rendert
  • Aanpassingen en/of uitbreidingen werken (zie ook Testing components)
  • Visual regression test (door LUX)

Links

LUX

Utrecht focus border
Het is nu nog niet mogelijk om doormiddel van tokens een dikkere focus-border toe te voegen. Voorstel: dit accepteren en wachten op release met deze fix in utrecht code die LUX devs gaan doen op utrecht code.

@rwittenberns
Copy link
Contributor Author

rwittenberns commented Sep 24, 2024

Oplossen:
Focus border (width) token wordt niet overgenomen op utrecht component, devs van LUX doen hiervoor een PR op utrecht code.

Reactie van Robbert in Slack:
feat: add border-width/border-color tokens for textbox interaction states

Die border toevoegen heeft nogal wat voeten in de aarde, zie die PR. Ik ben nog niet helemaal tevreden over mijn aanpak in de PR, omdat dit niet werkt voor forced-colors mode. Zouden jullie misschien een tweede PR willen maken met een variant waar de extra border-width met calc() van de padding afgehaald wordt?
En als dit werkt moet dit voor alle form controls gebeuren, in elk geval ook textarea, select, listbox, maar liefst ook radio button en checkbox

@rwittenberns rwittenberns assigned remypar5 and MrSkippy and unassigned remypar5 Sep 24, 2024
@rwittenberns
Copy link
Contributor Author

LUX tokens eerst mergen, daarna kan deze opgepakt worden.

@MrSkippy MrSkippy mentioned this issue Oct 1, 2024
8 tasks
@MrSkippy MrSkippy added new component Adding a new component to the library react Issue involving a ReactJS component labels Oct 1, 2024
@MrSkippy MrSkippy removed their assignment Oct 8, 2024
@VladAfanasev VladAfanasev self-assigned this Oct 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
new component Adding a new component to the library react Issue involving a ReactJS component
Projects
Status: In code review
Development

No branches or pull requests

4 participants