Merge pull request #777 from spotify/rugvip/cleanup

packages/core: fix import and remove bonus code

packages/core/src/api/app/types.ts

@@ -15,7 +15,7 @@
  */
 
 import { ComponentType } from 'react';
-import { IconComponent, SystemIconKey } from 'icons';
+import { IconComponent, SystemIconKey } from '../../icons';
 
 export type App = {
   getSystemIcon(key: SystemIconKey): IconComponent;

packages/core/src/components/BasicStepper/BasicVerticalStep.tsx

@@ -1,86 +0,0 @@
-/*
- * Copyright 2020 Spotify AB
- *
- * Licensed under the Apache License, Version 2.0 (the "License");
- * you may not use this file except in compliance with the License.
- * You may obtain a copy of the License at
- *
- *     http://www.apache.org/licenses/LICENSE-2.0
- *
- * Unless required by applicable law or agreed to in writing, software
- * distributed under the License is distributed on an "AS IS" BASIS,
- * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
- * See the License for the specific language governing permissions and
- * limitations under the License.
- */
-import React, { FC } from 'react';
-import {
-  Step as MuiStep,
-  StepContent,
-  StepLabel,
-  Typography,
-  makeStyles,
-} from '@material-ui/core';
-import SimpleStepperFooter from './SimpleStepperFooter';
-
-const useStyles = makeStyles((theme) => ({
-  end: {
-    padding: theme.spacing(3),
-  },
-}));
-
-export type StepActions = {
-  showNext?: boolean;
-  canNext?: () => boolean;
-  onNext?: () => void;
-  nextStep?: (current: number, last: number) => number;
-  nextText?: string;
-
-  showBack?: boolean;
-  backText?: string;
-  onBack?: () => void;
-
-  showRestart?: boolean;
-  canRestart?: () => boolean;
-  onRestart?: () => void;
-  restartText?: string;
-};
-
-export type StepProps = {
-  title: string;
-  children: React.ReactElement;
-  end?: boolean;
-  actions?: StepActions;
-};
-
-const Step: FC<StepProps> = ({
-  title,
-  children,
-  end,
-  actions,
-  ...muiProps
-}) => {
-  const classes = useStyles();
-
-  // The end step is not a part of the stepper
-  // It simply is the final screen with an option to have buttons such as reset or back
-  return end ? (
-    <div className={classes.end}>
-      <Typography variant="h6">{title}</Typography>
-      {children}
-      <SimpleStepperFooter actions={{ ...(actions || {}), showNext: false }} />
-    </div>
-  ) : (
-    <MuiStep {...muiProps}>
-      <StepLabel>
-        <Typography variant="h6">{title}</Typography>
-      </StepLabel>
-      <StepContent>
-        {children}
-        <SimpleStepperFooter actions={actions} />
-      </StepContent>
-    </MuiStep>
-  );
-};
-
-export default Step;