feat(Tooltip): adds triggerClass prop

innovaccer · Jul 24, 2020 · b5cfc67 · b5cfc67
1 parent e396caa
commit b5cfc67
Show file tree

Hide file tree

Showing 4 changed files with 11 additions and 20 deletions.
diff --git a/core/components/atoms/tooltip/Tooltip.tsx b/core/components/atoms/tooltip/Tooltip.tsx
@@ -1,7 +1,6 @@
 import * as React from 'react';
 import { Omit } from 'utility-types';
 import { PopperWrapper } from '@/utils';
-import classNames from 'classnames';
 
 export type PositionType =
   'top'
@@ -33,14 +32,13 @@ export interface TooltipProps extends DivProps {
    * @default true
    */
   appendToBody?: boolean;
+  /**
+   * Classes to be added to PopperWrapper trigger
+   */
+  triggerClass?: string;
 }
 
 interface IState {
-  position: {
-    top: number;
-    left: number;
-  };
-  style: React.CSSProperties;
   open: boolean;
 }
 
@@ -57,11 +55,6 @@ export class Tooltip extends React.Component<TooltipProps, IState> {
     super(props);
 
     this.state = {
-      position: {
-        top: 0,
-        left: 0,
-      },
-      style: {},
       open: false
     };
   }
@@ -81,18 +74,14 @@ export class Tooltip extends React.Component<TooltipProps, IState> {
       tooltip,
       children,
       className,
+      triggerClass,
       ...props
     } = this.props;
 
-    const classes = classNames({
-      ['Tooltip']: true
-    }, className);
-
     const tooltipWrapper = (
       <div
-        className={classes}
+        className="Tooltip"
         {...props}
-        style={this.state.style}
       >
         {tooltip}
       </div>
@@ -102,12 +91,12 @@ export class Tooltip extends React.Component<TooltipProps, IState> {
       <PopperWrapper
         trigger={children}
         placement={this.props.position}
-        style={this.state.style}
         appendToBody={appendToBody}
         on={'hover'}
         offset={'Medium'}
         onToggle={this.onToggle}
         open={this.state.open}
+        triggerClass={triggerClass}
       >
         {tooltipWrapper}
       </PopperWrapper>

diff --git a/core/components/molecules/popover/Popover.tsx b/core/components/molecules/popover/Popover.tsx
@@ -80,7 +80,7 @@ export interface PopoverProps extends BaseProps {
    */
   children: React.ReactNode;
   /**
-   * Classes to be added to PopperWrapper trigger
+   * Class to be added to PopperWrapper trigger
    */
   triggerClass?: string;
 }

diff --git a/core/utils/PopperWrapper.tsx b/core/utils/PopperWrapper.tsx
@@ -34,7 +34,7 @@ interface Props {
   triggerClass?: string;
   placement: PositionType;
   children: React.ReactElement<any>;
-  style: React.CSSProperties;
+  style?: React.CSSProperties;
   appendToBody: boolean;
   on?: actionType;
   offset: keyof typeof Offsets;

diff --git a/css/src/components/tooltip.css b/css/src/components/tooltip.css
@@ -8,5 +8,7 @@
   background: var(--inverse);
   overflow: hidden;
   box-sizing: border-box;
+  word-break: break-word;
+  hyphens: auto;
 }