Skip to content

Commit

Permalink
remove extra padding and global style
Browse files Browse the repository at this point in the history
  • Loading branch information
mattseddon committed Sep 8, 2022
1 parent 5d57f8b commit 4bdb869
Show file tree
Hide file tree
Showing 4 changed files with 1 addition and 12 deletions.
2 changes: 0 additions & 2 deletions webview/src/experiments/components/table/Indicators.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,6 @@ import {
} from '../../../shared/components/icons'
import { sendMessage } from '../../../shared/vscode'
import Tooltip from '../../../shared/components/tooltip/Tooltip'
import tooltipStyles from '../../../shared/components/tooltip/styles.module.scss'
import { pluralize } from '../../../util/strings'
import { ExperimentsState } from '../../store'

Expand All @@ -31,7 +30,6 @@ export const IndicatorTooltip: React.FC<IndicatorTooltipProps> = ({
placement="bottom-start"
disabled={!tooltipContent}
content={tooltipContent}
className={tooltipStyles.padded}
ref={wrapperRef}
>
{children}
Expand Down
2 changes: 0 additions & 2 deletions webview/src/shared/components/iconMenu/IconMenu.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,6 @@ import { useSingleton } from '@tippyjs/react'
import { IconMenuItem, IconMenuItemProps } from './IconMenuItem'
import styles from './styles.module.scss'
import Tooltip from '../tooltip/Tooltip'
import tooltipStyles from '../tooltip/styles.module.scss'

interface IconMenuProps {
items: IconMenuItemProps[]
Expand All @@ -29,7 +28,6 @@ export const IconMenu: React.FC<IconMenuProps> = ({ items }) => {

return (
<Tooltip
className={tooltipStyles.padded}
singleton={tooltipSource}
placement="bottom-end"
popperOptions={popperOptions}
Expand Down
4 changes: 1 addition & 3 deletions webview/src/shared/components/tooltip/Tooltip.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -28,8 +28,7 @@ const TooltipRenderFunction: React.ForwardRefRenderFunction<
onTrigger,
appendTo,
isContextMenu = false,
animation = false,
className = typeof content === 'string' ? styles.padded : undefined
animation = false
},
ref
) => (
Expand All @@ -44,7 +43,6 @@ const TooltipRenderFunction: React.ForwardRefRenderFunction<
<div className={styles.tooltipContent}>{content}</div>
)
}
className={className}
placement={placement}
delay={delay}
disabled={disabled}
Expand Down
5 changes: 0 additions & 5 deletions webview/src/shared/components/tooltip/styles.module.scss
Original file line number Diff line number Diff line change
Expand Up @@ -4,11 +4,6 @@ $foreground: var(--vscode-quickInput-foreground);
$background: var(--vscode-quickInput-background);
$font: var(--vscode-font-family);

:global(.tippy-box).padded {
& > :global(.tippy-content) {
padding: 2px 6px;
}
}
:global(.tippy-box) {
filter: $vscode-drop-shadow;
font-family: $font;
Expand Down

0 comments on commit 4bdb869

Please sign in to comment.