feat: get rid of view=component for text

core-ds · Feb 11, 2025 · 29f49f9 · 29f49f9
1 parent 165b14b
commit 29f49f9
Show file tree

Hide file tree

Showing 13 changed files with 15 additions and 38 deletions.
diff --git a/.changeset/pretty-ties-hear.md b/.changeset/pretty-ties-hear.md
@@ -0,0 +1,5 @@
+---
+'@alfalab/core-components-typography': major
+---
+
+Удален тип пропса `view` - `component`. Используйте `component-primary`
diff --git a/packages/file-upload-item/src/__snapshots__/Component.test.tsx.snap b/packages/file-upload-item/src/__snapshots__/Component.test.tsx.snap
@@ -99,7 +99,7 @@ Object {
           class="container"
         >
           <span
-            class="title primary component"
+            class="title primary component-primary"
           >
             title
           </span>
@@ -210,7 +210,7 @@ Object {
         class="container"
       >
         <span
-          class="title primary component"
+          class="title primary component-primary"
         >
           title
         </span>

diff --git a/packages/file-upload-item/src/components/content/content.tsx b/packages/file-upload-item/src/components/content/content.tsx
@@ -34,7 +34,7 @@ export const Content = () => {
                         [styles.truncate]: truncate,
                         [styles.restore]: showRestore,
                     })}
-                    view='component'
+                    view='component-primary'
                     color='primary'
                 >
                     {title}

diff --git a/packages/gallery/src/components/info-bar/Component.tsx b/packages/gallery/src/components/info-bar/Component.tsx
@@ -59,7 +59,7 @@ export const InfoBar = () => {
         <Text
             className={styles.description}
             tag='div'
-            view='component'
+            view='component-primary'
             color='static-primary-light'
         >
             {image?.name}

diff --git a/packages/modal/src/__snapshots__/Component.test.tsx.snap b/packages/modal/src/__snapshots__/Component.test.tsx.snap
@@ -187,7 +187,7 @@ exports[`ModalMobile snapshots tests should match snapshot 1`] = `
                       </svg>
                     </div>
                     <span
-                      class="text component medium"
+                      class="text component-primary medium"
                       style="opacity: 1;"
                     >
                       Назад

diff --git a/packages/navigation-bar-private/src/components/back-arrow-addon/Component.tsx b/packages/navigation-bar-private/src/components/back-arrow-addon/Component.tsx
@@ -66,7 +66,7 @@ export const BackArrowAddon: React.FC<BackArrowAddonProps> = ({
                 {textOpacity > 0 && text && (
                     <Text
                         className={styles.text}
-                        view={view === 'desktop' ? 'primary-large' : 'component'}
+                        view={view === 'desktop' ? 'primary-large' : 'component-primary'}
                         weight='medium'
                         style={{ opacity: textOpacity }}
                     >

diff --git a/packages/navigation-bar/src/components/action-text-addon/Component.tsx b/packages/navigation-bar/src/components/action-text-addon/Component.tsx
@@ -52,7 +52,7 @@ export const ActionTextAddon: React.FC<ActionTextAddonProps> = ({
     >
         <Text
             className={styles.text}
-            view='component'
+            view='component-primary'
             weight={action === 'primary' ? 'medium' : 'regular'}
         >
             {text}

diff --git a/packages/pure-cell/src/components/amount/component.tsx b/packages/pure-cell/src/components/amount/component.tsx
@@ -19,7 +19,7 @@ type Props = {
     /**
      * Размер денежного значения
      */
-    textView?: 'component' | 'primary-small';
+    textView?: 'component-primary' | 'primary-small';
 
     /**
      * Цвет денежного значения
@@ -40,7 +40,7 @@ type Props = {
 
 export const Amount: React.FC<Props> = ({
     weight = 'normal',
-    textView = 'component',
+    textView = 'component-primary',
     color = 'primary',
     minority,
     minorUnits = 100,

diff --git a/packages/pure-cell/src/components/text/component.tsx b/packages/pure-cell/src/components/text/component.tsx
@@ -26,7 +26,6 @@ type Props = {
         | 'primary-large'
         | 'primary-medium'
         | 'primary-small'
-        | 'component'
         | 'component-primary'
         | 'secondary-large';
 

diff --git a/packages/side-panel/src/__snapshots__/Component.test.tsx.snap b/packages/side-panel/src/__snapshots__/Component.test.tsx.snap
@@ -194,7 +194,7 @@ exports[`SidePanelMobile snapshots tests should match snapshot 1`] = `
                         </svg>
                       </div>
                       <span
-                        class="text component medium"
+                        class="text component-primary medium"
                         style="opacity: 1;"
                       >
                         Назад

diff --git a/packages/typography/src/text/component.test.tsx b/packages/typography/src/text/component.test.tsx
@@ -44,7 +44,6 @@ describe('Text', () => {
                 'secondary-large',
                 'secondary-medium',
                 'secondary-small',
-                'component',
                 'caps',
                 'tagline',
             ];

diff --git a/packages/typography/src/text/component.tsx b/packages/typography/src/text/component.tsx
@@ -22,7 +22,6 @@ type TextBaseProps = {
         | 'secondary-large'
         | 'secondary-medium'
         | 'secondary-small'
-        | 'component'
         | 'component-primary'
         | 'component-secondary'
         | 'caps'
@@ -91,23 +90,6 @@ type TextPTagProps = Omit<TextBaseProps, 'tag' | 'defaultMargins'> & {
 
 export type TextProps = Omit<NativeProps, 'color'> & (TextBaseProps | TextPTagProps);
 
-const logWarning = (view: Required<TextBaseProps>['view']) => {
-    if (process.env.NODE_ENV !== 'development') {
-        return;
-    }
-
-    const viewsMap: { [key: string]: string } = {
-        component: 'component-primary',
-    };
-
-    // eslint-disable-next-line no-console
-    console.warn(
-        // eslint-disable-next-line prefer-template
-        `@alfalab/core-components/typography: view='${view}' будет удален в следующих мажорных версиях. ` +
-            `Используйте view='${viewsMap[view]}'.`,
-    );
-};
-
 export const Text = forwardRef<TextElementType, TextProps>(
     (
         {
@@ -127,10 +109,6 @@ export const Text = forwardRef<TextElementType, TextProps>(
         },
         ref,
     ) => {
-        if (view === 'component') {
-            logWarning(view);
-        }
-
         const { renderSkeleton, textRef } = useSkeleton(showSkeleton, skeletonProps);
 
         const skeleton = renderSkeleton({

diff --git a/packages/typography/src/text/index.module.css b/packages/typography/src/text/index.module.css
@@ -39,10 +39,6 @@
     @mixin paragraph_secondary_small;
 }
 
-.component {
-    @mixin paragraph_component;
-}
-
 .component-primary {
     @mixin paragraph_component_primary;
 }