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

feat(plasma-tokens-utils): Update formatting tokens name from camelCase to kebab-case #971

Merged
merged 3 commits into from
Jan 17, 2024

Conversation

neretin-trike
Copy link
Collaborator

@neretin-trike neretin-trike commented Jan 16, 2024

Tokens naming

  • изменена нотация с camelCase на kebab-case для старых токенов цветов и типографики

Было:
Screenshot 2024-01-16 at 19 09 54

Стало:
Screenshot 2024-01-16 at 19 09 17

What/why changed

Библиотека styled-component в 5 версии автоматически переименовывала токены в kebab-case нотацию. Но новая 6 версия, а также другая css-in-js библиотека Linaria этого не делает. Поэтому для их поддержки это было сделано на нашей стороне

🐤 Download canary assets:
borderRadius
borderRadius_default_react-native--canary.971.7542106012.ts
borderRadius_plasma_b2c_react-native--canary.971.7542106012.ts
borderRadius_plasma_web_react-native--canary.971.7542106012.ts
borderRadius_sberHealth_react-native--canary.971.7542106012.ts
borderRadius_sbermarket_business_react-native--canary.971.7542106012.ts
borderRadius_sbermarket_metro_react-native--canary.971.7542106012.ts
borderRadius_sbermarket_react-native--canary.971.7542106012.ts
borderRadius_sbermarket_selgros_react-native--canary.971.7542106012.ts
borderRadius_sbermarket_wlbusiness_react-native--canary.971.7542106012.ts
borderRadius_sberonline_react-native--canary.971.7542106012.ts
borderRadius_sberprime_react-native--canary.971.7542106012.ts
borderRadius_stylesSalute_react-native--canary.971.7542106012.ts
color
color_default_ios-swift--canary.971.7542106012.swift
color_default_kotlin--canary.971.7542106012.kt
color_default_react-native--canary.971.7542106012.ts
color_default_xml--canary.971.7542106012.xml
color_plasma_b2c_ios-swift--canary.971.7542106012.swift
color_plasma_b2c_kotlin--canary.971.7542106012.kt
color_plasma_b2c_react-native--canary.971.7542106012.ts
color_plasma_b2c_xml--canary.971.7542106012.xml
color_plasma_web_ios-swift--canary.971.7542106012.swift
color_plasma_web_kotlin--canary.971.7542106012.kt
color_plasma_web_react-native--canary.971.7542106012.ts
color_plasma_web_xml--canary.971.7542106012.xml
color_sberHealth_ios-swift--canary.971.7542106012.swift
color_sberHealth_kotlin--canary.971.7542106012.kt
color_sberHealth_react-native--canary.971.7542106012.ts
color_sberHealth_xml--canary.971.7542106012.xml
color_sbermarket_business_ios-swift--canary.971.7542106012.swift
color_sbermarket_business_kotlin--canary.971.7542106012.kt
color_sbermarket_business_react-native--canary.971.7542106012.ts
color_sbermarket_business_xml--canary.971.7542106012.xml
color_sbermarket_ios-swift--canary.971.7542106012.swift
color_sbermarket_kotlin--canary.971.7542106012.kt
color_sbermarket_metro_ios-swift--canary.971.7542106012.swift
color_sbermarket_metro_kotlin--canary.971.7542106012.kt
color_sbermarket_metro_react-native--canary.971.7542106012.ts
color_sbermarket_metro_xml--canary.971.7542106012.xml
color_sbermarket_react-native--canary.971.7542106012.ts
color_sbermarket_selgros_ios-swift--canary.971.7542106012.swift
color_sbermarket_selgros_kotlin--canary.971.7542106012.kt
color_sbermarket_selgros_react-native--canary.971.7542106012.ts
color_sbermarket_selgros_xml--canary.971.7542106012.xml
color_sbermarket_wlbusiness_ios-swift--canary.971.7542106012.swift
color_sbermarket_wlbusiness_kotlin--canary.971.7542106012.kt
color_sbermarket_wlbusiness_react-native--canary.971.7542106012.ts
color_sbermarket_wlbusiness_xml--canary.971.7542106012.xml
color_sbermarket_xml--canary.971.7542106012.xml
color_sberonline_ios-swift--canary.971.7542106012.swift
color_sberonline_kotlin--canary.971.7542106012.kt
color_sberonline_react-native--canary.971.7542106012.ts
color_sberonline_xml--canary.971.7542106012.xml
color_sberprime_ios-swift--canary.971.7542106012.swift
color_sberprime_kotlin--canary.971.7542106012.kt
color_sberprime_react-native--canary.971.7542106012.ts
color_sberprime_xml--canary.971.7542106012.xml
color_stylesSalute_ios-swift--canary.971.7542106012.swift
color_stylesSalute_kotlin--canary.971.7542106012.kt
color_stylesSalute_react-native--canary.971.7542106012.ts
color_stylesSalute_xml--canary.971.7542106012.xml
shadow
shadow_default_react-native--canary.971.7542106012.ts
shadow_plasma_b2c_react-native--canary.971.7542106012.ts
shadow_plasma_web_react-native--canary.971.7542106012.ts
shadow_sberHealth_react-native--canary.971.7542106012.ts
shadow_sbermarket_business_react-native--canary.971.7542106012.ts
shadow_sbermarket_metro_react-native--canary.971.7542106012.ts
shadow_sbermarket_react-native--canary.971.7542106012.ts
shadow_sbermarket_selgros_react-native--canary.971.7542106012.ts
shadow_sbermarket_wlbusiness_react-native--canary.971.7542106012.ts
shadow_sberonline_react-native--canary.971.7542106012.ts
shadow_sberprime_react-native--canary.971.7542106012.ts
shadow_stylesSalute_react-native--canary.971.7542106012.ts
spacing
spacing_default_react-native--canary.971.7542106012.ts
spacing_plasma_b2c_react-native--canary.971.7542106012.ts
spacing_plasma_web_react-native--canary.971.7542106012.ts
spacing_sberHealth_react-native--canary.971.7542106012.ts
spacing_sbermarket_business_react-native--canary.971.7542106012.ts
spacing_sbermarket_metro_react-native--canary.971.7542106012.ts
spacing_sbermarket_react-native--canary.971.7542106012.ts
spacing_sbermarket_selgros_react-native--canary.971.7542106012.ts
spacing_sbermarket_wlbusiness_react-native--canary.971.7542106012.ts
spacing_sberonline_react-native--canary.971.7542106012.ts
spacing_sberprime_react-native--canary.971.7542106012.ts
spacing_stylesSalute_react-native--canary.971.7542106012.ts
typo
typo_mage_ios-swift--canary.971.7542106012.swift
typo_mage_kotlin--canary.971.7542106012.kt
typo_mage_react-native--canary.971.7542106012.ts
typo_plasma_ios-swift--canary.971.7542106012.swift
typo_plasma_kotlin--canary.971.7542106012.kt
typo_plasma_react-native--canary.971.7542106012.ts
typo_ruler_ios-swift--canary.971.7542106012.swift
typo_ruler_kotlin--canary.971.7542106012.kt
typo_ruler_react-native--canary.971.7542106012.ts
typo_sage_ios-swift--canary.971.7542106012.swift
typo_sage_kotlin--canary.971.7542106012.kt
typo_sage_react-native--canary.971.7542106012.ts
typo_sbermarket_ios-swift--canary.971.7542106012.swift
typo_sbermarket_kotlin--canary.971.7542106012.kt
typo_sbermarket_react-native--canary.971.7542106012.ts
typo_soulmate_ios-swift--canary.971.7542106012.swift
typo_soulmate_kotlin--canary.971.7542106012.kt
typo_soulmate_react-native--canary.971.7542106012.ts
📦 Published PR as canary version: Canary Versions

✨ Test out this PR locally via:

npm install @salutejs/[email protected]
npm install @salutejs/[email protected]
npm install @salutejs/[email protected]
npm install @salutejs/[email protected]
npm install @salutejs/[email protected]
npm install @salutejs/[email protected]
npm install @salutejs/[email protected]
npm install @salutejs/[email protected]
npm install @salutejs/[email protected]
npm install @salutejs/[email protected]
npm install @salutejs/[email protected]
npm install @salutejs/[email protected]
npm install @salutejs/[email protected]
npm install @salutejs/[email protected]
npm install @salutejs/[email protected]
npm install @salutejs/[email protected]
# or 
yarn add @salutejs/[email protected]
yarn add @salutejs/[email protected]
yarn add @salutejs/[email protected]
yarn add @salutejs/[email protected]
yarn add @salutejs/[email protected]
yarn add @salutejs/[email protected]
yarn add @salutejs/[email protected]
yarn add @salutejs/[email protected]
yarn add @salutejs/[email protected]
yarn add @salutejs/[email protected]
yarn add @salutejs/[email protected]
yarn add @salutejs/[email protected]
yarn add @salutejs/[email protected]
yarn add @salutejs/[email protected]
yarn add @salutejs/[email protected]
yarn add @salutejs/[email protected]

@Salute-Eva
Copy link
Contributor

Theme Builder app deployed!

http://plasma.sberdevices.ru/pr/plasma-theme-builder-pr-971/

'--plasma-typo-headline2-font-size': '1.5rem',
'--plasma-typo-headline2-letter-spacing': '-0.0220em',
'--plasma-typo-headline2-line-height': '1.75rem',
'--plasma-typo-headline3-font-family': '"SB Sans Text","SBSansText","Helvetica","Arial",sans-serif',
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@neretin-trike
А напомни пожалуйста на что влияет нейминг?

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

в описании как раз есть это :) Линария и другие библиотеки (даже тот же styled-component 6) внутри себя не превращают токены в эту нотацию. А это в свою очередь будет ломаться, т.к. у нас везде используется kebab-case

@neretin-trike neretin-trike added this pull request to the merge queue Jan 17, 2024
Merged via the queue into dev with commit 8b454fa Jan 17, 2024
27 checks passed
@neretin-trike neretin-trike deleted the neretinaa/add-kebab-formatting-for-tokens branch January 17, 2024 15:17
@Salute-Eva
Copy link
Contributor

🚀 This PR is included in version: @salutejs/[email protected], @salutejs/[email protected], @salutejs/[email protected], @salutejs/[email protected], @salutejs/[email protected], @salutejs/[email protected], @salutejs/[email protected], @salutejs/[email protected], @salutejs/[email protected], @salutejs/[email protected], @salutejs/[email protected], @salutejs/[email protected], @salutejs/[email protected], @salutejs/[email protected], @salutejs/[email protected], @salutejs/[email protected], @salutejs/[email protected], @salutejs/[email protected], @salutejs/[email protected], @salutejs/[email protected], @salutejs/[email protected] 🚀

@Salute-Eva
Copy link
Contributor

🚀 This PR is included in version: @salutejs/[email protected], @salutejs/[email protected], @salutejs/[email protected], @salutejs/[email protected], @salutejs/[email protected], @salutejs/[email protected], @salutejs/[email protected], @salutejs/[email protected], @salutejs/[email protected], @salutejs/[email protected], @salutejs/[email protected], @salutejs/[email protected], @salutejs/[email protected], @salutejs/[email protected], @salutejs/[email protected], @salutejs/[email protected], @salutejs/[email protected], @salutejs/[email protected], @salutejs/[email protected], @salutejs/[email protected], @salutejs/[email protected] 🚀

@Salute-Eva
Copy link
Contributor

🚀 This PR is included in version: @salutejs/[email protected], @salutejs/[email protected], @salutejs/[email protected], @salutejs/[email protected], @salutejs/[email protected], @salutejs/[email protected], @salutejs/[email protected], @salutejs/[email protected], @salutejs/[email protected], @salutejs/[email protected], @salutejs/[email protected], @salutejs/[email protected], @salutejs/[email protected], @salutejs/[email protected], @salutejs/[email protected], @salutejs/[email protected], @salutejs/[email protected], @salutejs/[email protected], @salutejs/[email protected], @salutejs/[email protected], @salutejs/[email protected] 🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants