Skip to content

Commit

Permalink
Revert "Merge pull request #23635 from storybookjs/charles-update-con…
Browse files Browse the repository at this point in the history
…trols-addon"

This reverts commit 0005430, reversing
changes made to a70dcab.
  • Loading branch information
ndelangen committed Aug 21, 2023
1 parent 3bbea20 commit df5b104
Show file tree
Hide file tree
Showing 15 changed files with 14 additions and 220 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -31,10 +31,6 @@ const getAbsolutePath = <I extends string>(input: I): I =>
dirname(require.resolve(join(input, 'package.json'))) as any;

const storybookPaths: Record<string, string> = {
// this is a temporary hack to get webpack to alias this correctly
[`@storybook/components/experimental`]: `${getAbsolutePath(
`@storybook/components`
)}/dist/experimental`,
...[
// these packages are not pre-bundled because of react dependencies
'components',
Expand Down
2 changes: 1 addition & 1 deletion code/ui/blocks/src/components/ArgsTable/ArgControl.tsx
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
import type { FC } from 'react';
import React, { useCallback, useState, useEffect } from 'react';

import { Link } from '@storybook/components/experimental';
import { Link } from '@storybook/components';
import {
BooleanControl,
ColorControl,
Expand Down
3 changes: 1 addition & 2 deletions code/ui/blocks/src/components/ArgsTable/ArgsTable.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -5,8 +5,7 @@ import { styled } from '@storybook/theming';
import { transparentize } from 'polished';
import { includeConditionalArg } from '@storybook/csf';
import { once } from '@storybook/client-logger';
import { IconButton, Icons, ResetWrapper } from '@storybook/components';
import { Link } from '@storybook/components/experimental';
import { IconButton, Icons, ResetWrapper, Link } from '@storybook/components';

import { ArgRow } from './ArgRow';
import { SectionRow } from './SectionRow';
Expand Down
6 changes: 3 additions & 3 deletions code/ui/blocks/src/components/ArgsTable/Empty.tsx
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
import type { FC } from 'react';
import React, { useEffect, useState } from 'react';
import { styled } from '@storybook/theming';
import { Link } from '@storybook/components/experimental';
import { Icons, Link } from '@storybook/components';

interface EmptyProps {
inAddonPanel?: boolean;
Expand Down Expand Up @@ -85,8 +85,8 @@ export const Empty: FC<EmptyProps> = ({ inAddonPanel }) => {
<Links>
{inAddonPanel && (
<>
<Link href="https://youtu.be/0gOfS6K0x0E" target="_blank" icon="Video" withArrow>
Watch 5m video
<Link href="https://youtu.be/0gOfS6K0x0E" target="_blank" withArrow>
<Icons icon="video" /> Watch 5m video
</Link>
<Divider />
<Link
Expand Down
4 changes: 2 additions & 2 deletions code/ui/components/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -52,7 +52,7 @@
"dist/html.d.ts"
],
"experimental": [
"dist/experimental.d.ts"
"dist/experimantal.d.ts"
]
}
},
Expand Down Expand Up @@ -105,7 +105,7 @@
"bundler": {
"entries": [
"./src/index.ts",
"./src/experimental.ts"
"./src/experimantal.ts"
],
"platform": "neutral"
},
Expand Down
6 changes: 0 additions & 6 deletions code/ui/components/src/experimental.ts
Original file line number Diff line number Diff line change
Expand Up @@ -8,9 +8,3 @@
*/

export { Button } from './new/Button/Button';
export { Input } from './new/Input/Input';
export { Select } from './new/Select/Select';
export { Link } from './new/Link/Link';
export { Icon } from './new/Icon/Icon';
export { IconButton } from './new/IconButton/IconButton';
export { Toolbar } from './new/Toolbar/Toolbar';
4 changes: 0 additions & 4 deletions code/ui/components/src/new/Button/Button.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,6 @@ import { styled } from '@storybook/theming';
import { darken, lighten, rgba, transparentize } from 'polished';
import type { Icons } from '@storybook/icons';
import type { PropsOf } from '../utils/types';
import { Icon } from '../Icon/Icon';

interface ButtonProps<T extends React.ElementType = React.ElementType> {
children: string;
Expand All @@ -23,11 +22,8 @@ export const Button: {
displayName?: string;
} = forwardRef(
({ as, children, icon, ...props }: ButtonProps, ref: React.Ref<HTMLButtonElement>) => {
const LocalIcon = Icon[icon];

return (
<StyledButton as={as} ref={ref} {...props}>
{icon && <LocalIcon />}
{children}
</StyledButton>
);
Expand Down
1 change: 0 additions & 1 deletion code/ui/components/src/new/Icon/Icon.tsx

This file was deleted.

103 changes: 0 additions & 103 deletions code/ui/components/src/new/Link/Link.stories.tsx

This file was deleted.

76 changes: 0 additions & 76 deletions code/ui/components/src/new/Link/Link.tsx

This file was deleted.

9 changes: 7 additions & 2 deletions code/ui/components/src/new/Select/SelectItem.tsx
Original file line number Diff line number Diff line change
@@ -1,20 +1,25 @@
import * as RadixSelect from '@radix-ui/react-select';
import React, { forwardRef } from 'react';
import { styled } from '@storybook/theming';
import { Icon } from '@storybook/components/experimental';

interface SelectItemProps {
children: React.ReactNode;
value: string;
}

const FakeIcon = styled.div(({ theme }) => ({
width: 12,
height: 12,
backgroundColor: theme.color.mediumdark,
}));

export const SelectItem = forwardRef<HTMLDivElement, SelectItemProps>(
({ children, ...props }, forwardedRef) => {
return (
<StyledItem {...props} ref={forwardedRef}>
<RadixSelect.ItemText>{children}</RadixSelect.ItemText>
<StyledItemIndicator className="SelectItemIndicator">
<Icon.FaceHappy />
<FakeIcon />
</StyledItemIndicator>
</StyledItem>
);
Expand Down
9 changes: 0 additions & 9 deletions code/ui/manager/src/globals/exports.ts
Original file line number Diff line number Diff line change
Expand Up @@ -114,15 +114,6 @@ export default {
'resetComponents',
'withReset',
],
'@storybook/components/experimental': [
'Button',
'Icon',
'IconButton',
'Input',
'Link',
'Select',
'Toolbar',
],
'@storybook/channels': [
'Channel',
'PostMessageTransport',
Expand Down
2 changes: 0 additions & 2 deletions code/ui/manager/src/globals/runtime.ts
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,6 @@ import * as REACT from 'react';
import * as REACTDOM from 'react-dom';

import * as STORYBOOKCOMPONENTS from '@storybook/components';
import * as STORYBOOKCOMPONENTSEXPERIMENTAL from '@storybook/components/experimental';
import * as STORYBOOKCHANNELS from '@storybook/channels';
import * as STORYBOOKEVENTS from '@storybook/core-events';
import * as STORYBOOKROUTER from '@storybook/router';
Expand All @@ -17,7 +16,6 @@ export const values: Required<Record<keyof typeof Keys, any>> = {
react: REACT as any,
'react-dom': REACTDOM,
'@storybook/components': STORYBOOKCOMPONENTS,
'@storybook/components/experimental': STORYBOOKCOMPONENTSEXPERIMENTAL,
'@storybook/channels': STORYBOOKCHANNELS,
'@storybook/core-events': STORYBOOKEVENTS,
'@storybook/router': STORYBOOKROUTER,
Expand Down
1 change: 0 additions & 1 deletion code/ui/manager/src/globals/types.ts
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,6 @@ export enum Keys {
'react' = '__REACT__',
'react-dom' = '__REACTDOM__',
'@storybook/components' = '__STORYBOOKCOMPONENTS__',
'@storybook/components/experimental' = '__STORYBOOKCOMPONENTSEXPERIMENTAL__',
'@storybook/channels' = '__STORYBOOKCHANNELS__',
'@storybook/core-events' = '__STORYBOOKCOREEVENTS__',
'@storybook/router' = '__STORYBOOKROUTER__',
Expand Down
4 changes: 0 additions & 4 deletions scripts/verdaccio.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -31,10 +31,6 @@ packages:
access: $all
publish: $all
proxy: npmjs
'@storybook/icons':
access: $all
publish: $all
proxy: npmjs
'@storybook/global':
access: $all
publish: $all
Expand Down

0 comments on commit df5b104

Please sign in to comment.