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

[Theme] подготовка к 8px сетке #1991

Merged
merged 117 commits into from
Jul 17, 2020
Merged

[Theme] подготовка к 8px сетке #1991

merged 117 commits into from
Jul 17, 2020

Conversation

rustho
Copy link
Contributor

@rustho rustho commented Apr 13, 2020

Общий PR для всех работ по добавлению новых переменных.
#1733

переменные

Компонент Добавлены Пересвязаны с глобальными Добавлена внутренняя связь
Token tokenMargin tokenLineHeight tokenPaddingX tokenMarginBeforeIcon tokenBorderRadius tokenFontSize tokenTextColorDisabled tokenDefaultIdle tokenDefaultActive tokenGrayIdle tokenGrayActive tokenBlueIdle tokenBlueActive tokenGreenIdle tokenGreenActive tokenYellowIdle tokenYellowActive tokenRedIdle tokenRedActive tokenWhite tokenBlack tokenBorderColorWarning tokenBorderColorError
Toggle toggleSwitcherSize toggleSwitcherBorderRadius toggleHeight toggleWidth toggleBorderRadius toggleBorderWidth
Input inputWidth inputTextColorDisabled inputLineHeightSmall inputLineHeightMedium inputLineHeightLarge inputHeightSmall inputHeightMedium inputHeightLarge inputPaddingYSmall inputPaddingYMedium inputPaddingYLarge inputPaddingX inputPaddingBetweenIcon inputBorderColor inputBorderColorFocus inputBorderColorError inputBorderColorWarning inputPlaceholderColor inputPlaceholderColorLight inputBlinkColor inputBorderWidth
Checkbox checkboxLineHeight checkboxBoxSize checkboxFontSize checkboxPaddingX chb->checkbox checkboxTextColorDefault checkboxTextColorDisabled checkboxOutlineColorFocus checkboxBorderColorWarning checkboxBorderColorError checkboxBgDisabled
Textarea textareaPaddingY textareaPaddingX textareaFontSizeMedium textareaBorderColor textareaHeightSmall textareaBorderColorFocus textareaBorderColorFocus textareaPlaceholderColorLight textareaTextColorDisabled textareaPlaceholderColor textareaBorderColorError textareaBorderColorError textareaBorderColorError textareaBorderColorError textareaBorderColorWarning textareaBorderColorWarning textareaBorderColorWarning textareaBorderColorWarning
Radio radioBorderWidthFocus radioBorderWidthFocus radioSizeAfter radioSizeAfter radioBorderWidthFocus radioBulletSize radioBulletSize radioLineHeight radioPaddingX radioBorderColorFocus radioBorderColorWarning radioBorderColorError
Tab tabsMarginX tabBorderWidth tabFontSize tabLineHeight tabPaddingX tabPaddingX tabPaddingY tabBorderWidth tabBorderWidthFocus tabTextColorDefault tabBorderWidth tabPaddingX
Spinner spinnerColor spinnerPaddingXSmall spinnerFontSizeSmall spinnerLineHeightSmall spinnerFontSizeMedium spinnerLineHeightMedium spinnerPaddingYMedium spinnerFontSizeLarge spinnerLineHeightLarge spinnerPaddingYLarge

rustho added 30 commits April 7, 2020 16:32
@zhzz
Copy link
Member

zhzz commented Jul 14, 2020

  • Хорошо бы вынести устаревшие переменные в отдельный список и пометить как-то. А то в редакторе сейчас все перемешано.

image

@zhzz
Copy link
Member

zhzz commented Jul 14, 2020

  • Еще кандидат на выброс: controlBorderRadius. Используется только в кнопке и в селекте (тоже по сути кнопка).

@zhzz
Copy link
Member

zhzz commented Jul 16, 2020

  • Починить стили borderless для Input
  • Поправить кнопки-стрелки размеров medium и large
  • Настроить отступы в Spinner
  • Уменьшить горизонтальный padding в Token
  • Проверить отступ и размер стрелки в Select
    В Select легаси-верстка, стрелка пока останется не по макетам
  • Переименовать borderWidthFocus в outlineWidth?

@zhzz
Copy link
Member

zhzz commented Jul 17, 2020

  • Поправил отступы в Tab
  • Переименовал spinnerPadding(...) в spinnerCaptionGap(...)
  • Переименовал tokenCloseIcon(...) в tokenRemoveIcon(...)
  • Добавил tokenRemoveIconBoxSizing

@zhzz zhzz merged commit f40e102 into master Jul 17, 2020
@zhzz zhzz deleted the theme-8px/common branch July 17, 2020 10:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

4 participants