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

Checkbox - remove gap to the left #1156

Open
3 of 4 tasks
Tracked by #452
thrbnhrtmnn opened this issue Sep 11, 2024 · 1 comment
Open
3 of 4 tasks
Tracked by #452

Checkbox - remove gap to the left #1156

thrbnhrtmnn opened this issue Sep 11, 2024 · 1 comment
Labels
⭕ core team issue This is for the core team and should not be done by contributors 🎨 design issue Task is for designers ⌨️ dev issue Task is for developers

Comments

@thrbnhrtmnn
Copy link
Contributor

thrbnhrtmnn commented Sep 11, 2024

Description / User story

When comparing design and code of the Checkbox a few misalignments can be seen:

  • In Code the checkbox seems to have a gap left to the control, which is not there in design. The gap to the left should be removed
  • The focus ring in design and code seem to have different values, but they look the same. In sizeVariant=MD the container is 24x24px in design but 16x16px in code. Also the border-radius of the focus ring seems to not be defined by a token in design, but just is 5px, whereas in code we apparently use the token for the checkbox control to calculate the border radius of the focus ring. I am afraid this approach would not work anymore when the control should not have a radius but the focus ring should have one.

Requirements / Prerequisites

  • none

Acceptance Criteria

  • The checkbox does not have a gap left of the control on code
  • The focus ring in design is fully tokenized
  • The focus ring in code uses all tokens that are defined in design to look the same as in design

Additional information

  • ...

Code of Conduct

  • I agree to follow this project's Code of Conduct
@thrbnhrtmnn thrbnhrtmnn added 🎨 design issue Task is for designers ⌨️ dev issue Task is for developers ⭕ core team issue This is for the core team and should not be done by contributors labels Sep 11, 2024
@thrbnhrtmnn
Copy link
Contributor Author

As decided today, the tokenizing of the focus layer in the checkbox component should be done separately and at a later point. This requires some design research.

@thrbnhrtmnn thrbnhrtmnn changed the title Checkbox - fix focus ring sizes and remove gap to the left Checkbox - remove gap to the left Oct 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
⭕ core team issue This is for the core team and should not be done by contributors 🎨 design issue Task is for designers ⌨️ dev issue Task is for developers
Projects
None yet
Development

No branches or pull requests

1 participant