[Emotion] Merge override SCSS into baseline SCSS

elastic / eui

Elastic UI Framework 🙌

https://eui.elastic.co/

Other

6.08k stars 826 forks source link

[Emotion] Merge override SCSS into baseline SCSS #7042

Closed cee-chen closed 2 months ago

cee-chen commented 1 year ago

### List of components with Amsterdam override SCSS
- [ ] EuiKeyPadMenuItem
- [ ] EuiDescriptionList
- [ ] EuiSideNavItem
- [ ] EuiHue
- [ ] EuiRadio, EuiCheckbox, EuiSwitch
- [ ] EuiComboBox
- [ ] EuiFormControlLayout
- [ ] EuiFormControlLayoutDelimited
- [ ] EuiMarkdownEditor
- [ ] EuiDataGrid

See https://github.com/elastic/eui/blob/main/src/themes/amsterdam/overrides

yaodada123 commented 2 months ago

Hello, I would like to contribute to the project on this issue. Now that I have successfully run the project locally and have found the EuiKeyPadMenuItem style file code, as shown in the figure, can you give me further hints as to what I should do to accomplish this task? QQ图片20240702164409

cee-chen commented 2 months ago

Hey @yaodada123 - apologies, this task is actually out of date at this point. EuiKeyPadMenuItem has already been converted to Emotion and as such no longer has any SCSS to update. I'm going to go ahead and close this issue out as not planned.