Merge pull request #90 from spotify/freben/heights

Fix heights and paddings

frontend/packages/app/src/components/SideBar/SideBar.tsx

@@ -152,7 +152,7 @@ const useStyles = makeStyles(theme => ({
     position: 'relative',
     overflow: 'visible',
     width: theme.spacing(7) + 1,
-    height: '100vh',
+    height: '100%',
   },
   drawer: {
     display: 'flex',

frontend/packages/core/src/components/DefaultEntityPage/DefaultEntityPage.tsx

@@ -6,7 +6,7 @@ import { Switch, Route, Redirect } from 'react-router-dom';
 import DefaultEntityPageHeader from '../DefaultEntityPageHeader';
 import DefaultEntityPageNavbar from '../DefaultEntityPageNavbar';
 
-const useStyles = makeStyles<Theme>(theme => ({
+const useStyles = makeStyles<Theme>({
   root: {
     display: 'grid',
     gridTemplateAreas: `
@@ -16,7 +16,6 @@ const useStyles = makeStyles<Theme>(theme => ({
     gridTemplateRows: 'auto 1fr',
     gridTemplateColumns: 'auto 1fr',
     minHeight: '100%',
-    paddingBottom: theme.spacing(3),
   },
   header: {
     gridArea: 'header',
@@ -27,7 +26,7 @@ const useStyles = makeStyles<Theme>(theme => ({
   content: {
     gridArea: 'content',
   },
-}));
+});
 
 const DefaultEntityPage: FC<EntityPageProps> = ({ navItems, views }) => {
   const classes = useStyles();

frontend/packages/core/src/components/DefaultEntityPageNavbar/DefaultEntityPageNavbar.tsx

@@ -12,7 +12,7 @@ const useStyles = makeStyles<Theme>({
     transitionTimingFunction: 'ease-in',
     backgroundColor: '#eeeeee',
     boxShadow: '0px 0 4px 0px rgba(0,0,0,0.35)',
-    height: '100vh',
+    height: '100%',
   },
   list: {
     padding: 0,