review: resolve comment

VKCOM · Nov 7, 2024 · 9a72112 · 9a72112
1 parent d90fbc6
commit 9a72112
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 18 deletions.
diff --git a/packages/vkui/src/components/Clickable/Clickable.module.css b/packages/vkui/src/components/Clickable/Clickable.module.css
@@ -1,20 +1,22 @@
 .realClickable {
   cursor: pointer;
+  -webkit-tap-highlight-color: transparent;
 }
 
-.resetButtonStyle {
+/* stylelint-disable selector-max-type */
+button.realClickable {
   appearance: none;
   user-select: none;
-  -webkit-tap-highlight-color: transparent;
   -webkit-touch-callout: none;
   -webkit-user-drag: none;
 }
 
-.resetLinkStyle {
+a.realClickable {
   appearance: none;
   -webkit-tap-highlight-color: transparent;
   text-decoration: none;
 }
+/* stylelint-enable selector-max-type */
 
 .host:focus,
 .host:focus-visible {

diff --git a/packages/vkui/src/components/Clickable/Clickable.tsx b/packages/vkui/src/components/Clickable/Clickable.tsx
@@ -175,16 +175,6 @@ function component<T>({
   return {};
 }
 
-const getUserAgentResetClassName = (Component?: React.ElementType) => {
-  if (Component === 'a') {
-    return styles.resetLinkStyle;
-  }
-  if (Component === 'button') {
-    return styles.resetButtonStyle;
-  }
-  return;
-};
-
 /**
  * Базовый кликабельный корневой компонент.
  *
@@ -204,11 +194,7 @@ export const Clickable = <T,>({
 }: ClickableProps<T>): React.ReactNode => {
   const commonProps = component(restProps);
   const isClickable = checkClickable(restProps);
-  const baseClassName = classNames(
-    baseClassNameProp,
-    getUserAgentResetClassName(commonProps.Component),
-    styles.host,
-  );
+  const baseClassName = classNames(baseClassNameProp, styles.host);
 
   if (isClickable) {
     return (