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

Update components to use new design tokens #9175

Closed
Tracked by #9159
tw15egan opened this issue Jul 12, 2021 · 3 comments · Fixed by #9323
Closed
Tracked by #9159

Update components to use new design tokens #9175

tw15egan opened this issue Jul 12, 2021 · 3 comments · Fixed by #9323

Comments

@tw15egan
Copy link
Member

tw15egan commented Jul 12, 2021

Component tokens will need to be updated to use the new tokens.

Notion doc: https://www.notion.so/carbondesignsystem/Final-List-of-new-tokens-fc5985c7331341baa6546064399d36da
Component specs: #8187
Previous Epic: #8148

The previous attempts can be used as a reference
#9064
#9065
#9066
#9067

List of SASS Component Refactors:

@andreancardona
Copy link
Contributor

andreancardona commented Jul 13, 2021

@tw15egan @dakahn

Since these are assigned to us this sprint. Going to start looking at the following:

  • Accordion ✅
  • Breadcrumb ✅
  • Button ✅
  • Checkbox ✅
  • Codesnippet ✅
  • Combobox (does not seem to be converted to sass)

@andreancardona
Copy link
Contributor

andreancardona commented Jul 13, 2021

Moving on to these:

  • Overflow menu ✅
  • Pagination ✅
  • Pagination nav ✅
  • Popover (hasn't been converted to sass)
  • Progress indicator ✅
  • Radio button ✅
  • Search ✅
  • Select ✅
  • Slider ✅
  • Structured list ✅
  • Tabs ✅

@tw15egan
Copy link
Member Author

tw15egan commented Jul 14, 2021

Going to work on porting over ComboBox, Dropdown, Listbox and Multiselect today, those will need to be updated to the new tokens once they are merged in (#9203)

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 a pull request may close this issue.

2 participants