Skip to content

Commit

Permalink
remove SVGPopover and SVGTooltip as they only set one prop (#2601)
Browse files Browse the repository at this point in the history
* remove SVGPopover and SVGTooltip as they only set one prop

* missed one
  • Loading branch information
giladgray authored Jun 15, 2018
1 parent 1b43e80 commit 7498c91
Show file tree
Hide file tree
Showing 3 changed files with 0 additions and 41 deletions.
19 changes: 0 additions & 19 deletions packages/core/src/components/popover/svgPopover.tsx

This file was deleted.

19 changes: 0 additions & 19 deletions packages/core/src/components/tooltip/svgTooltip.tsx

This file was deleted.

3 changes: 0 additions & 3 deletions packages/core/test/isotest.js
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,6 @@ const customProps = {
KeyCombo: { combo: "?" },
OverflowList: { items: [], overflowRenderer: () => null, visibleItemRenderer: () => null },
Overlay: { lazy: false, usePortal: false },
SVGTooltip: tooltipContent,
TagInput: { values: ["foo", "bar", "baz"] },
Tooltip: tooltipContent,
Toaster: { usePortal: false },
Expand All @@ -25,8 +24,6 @@ const popoverTarget = React.createElement("button");
const customChildren = {
Hotkeys: React.createElement(Core.Hotkey, customProps.Hotkey),
Popover: popoverTarget,
SVGPopover: popoverTarget,
SVGTooltip: popoverTarget,
Tabs: React.createElement(Core.Tab, { key: 1, id: 1, title: "Tab one" }),
Tooltip: popoverTarget,
Toaster: React.createElement(Core.Toast, { message: "Toast" }),
Expand Down

1 comment on commit 7498c91

@blueprint-bot
Copy link

Choose a reason for hiding this comment

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

remove SVGPopover and SVGTooltip as they only set one prop (#2601)

Preview: documentation | landing | table

Please sign in to comment.