From a60a222bfc40e97394ad8988f7dbbe6852389ba7 Mon Sep 17 00:00:00 2001
From: Ray Ventura <ray@1099policy.com>
Date: Tue, 25 Jun 2019 20:49:41 -0700
Subject: [PATCH] reskin documentation

---
 src/common-elements/fields-layout.ts              | 15 ++++++++++-----
 src/common-elements/fields.ts                     |  2 ++
 src/common-elements/panels.ts                     |  2 +-
 src/components/Endpoint/Endpoint.tsx              | 10 +---------
 src/components/Endpoint/styled.elements.ts        |  8 +++-----
 src/components/RequestSamples/RequestSamples.tsx  |  9 ++-------
 .../ResponseSamples/ResponseSamples.tsx           |  4 +---
 src/components/Responses/Response.tsx             | 12 ++----------
 src/components/Responses/ResponseTitle.tsx        | 11 +----------
 src/components/Responses/ResponsesList.tsx        | 10 ----------
 src/services/models/Response.ts                   |  2 +-
 src/theme.ts                                      |  8 ++++----
 12 files changed, 28 insertions(+), 65 deletions(-)

diff --git a/src/common-elements/fields-layout.ts b/src/common-elements/fields-layout.ts
index 84a1787d..3cf568d2 100644
--- a/src/common-elements/fields-layout.ts
+++ b/src/common-elements/fields-layout.ts
@@ -12,7 +12,7 @@ export const PropertiesTableCaption = styled.caption`
 
 export const PropertyCell = styled.td<{ kind?: string }>`
   border-left: 1px solid ${props => props.theme.schema.linesColor};
-  border-bottom: 1px solid #e3e8ee;
+  border-top: 1px solid #e3e8ee;
   box-sizing: border-box;
   position: relative;
   padding: 10px 10px 10px 0;
@@ -55,7 +55,7 @@ export const PropertyCell = styled.td<{ kind?: string }>`
   }
 
   tr.expanded & {
-    border-bottom: none;
+    border-top: none;
   }
 `;
 
@@ -68,7 +68,8 @@ export const PropertyNameCell = styled(PropertyCell)`
   line-height: 20px;
   white-space: nowrap;
   font-size: 0.929em;
-  font-family: ${props => props.theme.typography.headings.fontFamily};
+  font-weight: bold;
+  font-family: Courier,monospace;
 
   &.deprecated {
     ${deprecatedCss};
@@ -80,13 +81,13 @@ export const PropertyNameCell = styled(PropertyCell)`
 `;
 
 export const PropertyDetailsCell = styled.td`
-  border-bottom: 1px solid #e3e8ee;
+  border-top: 1px solid #e3e8ee;
   padding: 10px 0;
   width: ${props => props.theme.schema.defaultDetailsWidth};
   box-sizing: border-box;
 
   tr.expanded & {
-    border-bottom: none;
+    border-top: none;
   }
 `;
 
@@ -126,6 +127,10 @@ export const PropertiesTable = styled.table`
   border-spacing: 0;
   width: 100%;
 
+  tr:first-child td {
+    border-top: none;
+  }
+
   > tr {
     vertical-align: middle;
   }
diff --git a/src/common-elements/fields.ts b/src/common-elements/fields.ts
index 9ab3a929..933feb1b 100644
--- a/src/common-elements/fields.ts
+++ b/src/common-elements/fields.ts
@@ -18,6 +18,8 @@ export const ClickablePropertyNameCell = styled(PropertyNameCell)`
 
 export const FieldLabel = styled.span`
   vertical-align: middle;
+  text-transform: uppercase;
+  font-family: ${({ theme }) => theme.typography.headings.fontFamily};
   font-size: ${({ theme }) => theme.typography.code.fontSize};
   line-height: 20px;
 `;
diff --git a/src/common-elements/panels.ts b/src/common-elements/panels.ts
index 692d7d35..31b00350 100644
--- a/src/common-elements/panels.ts
+++ b/src/common-elements/panels.ts
@@ -15,7 +15,7 @@ export const MiddlePanel = styled.div`
 export const Section = styled.div.attrs(props => ({
   [SECTION_ATTR]: props.id,
 })) <{ underlined?: boolean }>`
-  padding: 0 0 ${props => props.theme.spacing.sectionVertical}px 0;
+  padding: 20px 0 ${props => props.theme.spacing.sectionVertical}px 0;
   border-bottom: 1px solid #e3e8ee;
   &:last-child {
     min-height: calc(100vh + 1px);
diff --git a/src/components/Endpoint/Endpoint.tsx b/src/components/Endpoint/Endpoint.tsx
index 00d7bda0..d3c25cc6 100644
--- a/src/components/Endpoint/Endpoint.tsx
+++ b/src/components/Endpoint/Endpoint.tsx
@@ -1,5 +1,4 @@
 import * as React from 'react';
-import { ShelfIcon } from '../../common-elements';
 import { OperationModel } from '../../services';
 import { Markdown } from '../Markdown/Markdown';
 import { OptionsContext } from '../OptionsProvider';
@@ -48,16 +47,9 @@ export class Endpoint extends React.Component<EndpointProps, EndpointState> {
       <OptionsContext.Consumer>
         {options => (
           <OperationEndpointWrap>
-            <EndpointInfo onClick={this.toggle} expanded={expanded} inverted={inverted}>
+            <EndpointInfo expanded={expanded} inverted={inverted}>
               <HttpVerb type={operation.httpVerb}> {operation.httpVerb}</HttpVerb>{' '}
               <ServerRelativeURL>{operation.path}</ServerRelativeURL>
-              <ShelfIcon
-                float={'right'}
-                color={inverted ? 'black' : 'white'}
-                size={'20px'}
-                direction={expanded ? 'up' : 'down'}
-                style={{ marginRight: '-25px' }}
-              />
             </EndpointInfo>
             <ServersOverlay expanded={expanded}>
               {operation.servers.map(server => (
diff --git a/src/components/Endpoint/styled.elements.ts b/src/components/Endpoint/styled.elements.ts
index f00a032b..00253218 100644
--- a/src/components/Endpoint/styled.elements.ts
+++ b/src/components/Endpoint/styled.elements.ts
@@ -3,7 +3,7 @@ import styled from '../../styled-components';
 export const OperationEndpointWrap = styled.div`
   cursor: pointer;
   position: relative;
-  margin-bottom: 5px;
+  margin: 60px 0px 0px 0px;
 `;
 
 export const ServerRelativeURL = styled.span`
@@ -17,8 +17,7 @@ export const ServerRelativeURL = styled.span`
 export const EndpointInfo = styled.div<{ expanded?: boolean; inverted?: boolean }>`
   padding: 10px 30px 10px ${props => (props.inverted ? '10px' : '20px')};
   border-radius: ${props => (props.inverted ? '0' : '4px 4px 0 0')};
-  background-color: ${props =>
-    props.inverted ? 'transparent' : props.theme.codeSample.backgroundColor};
+  background-color: #3c4257;
   display: flex;
   white-space: nowrap;
   align-items: center;
@@ -36,10 +35,9 @@ export const EndpointInfo = styled.div<{ expanded?: boolean; inverted?: boolean
 
 export const HttpVerb = styled.span.attrs((props: { type: string }) => ({
   className: `http-verb ${props.type}`,
-}))<{ type: string }>`
+})) <{ type: string }>`
   font-size: 0.929em;
   line-height: 20px;
-  background-color: ${(props: any) => props.theme.colors.http[props.type] || '#999999'};
   color: #ffffff;
   padding: 3px 10px;
   text-transform: uppercase;
diff --git a/src/components/RequestSamples/RequestSamples.tsx b/src/components/RequestSamples/RequestSamples.tsx
index 66a4db53..4bea8781 100644
--- a/src/components/RequestSamples/RequestSamples.tsx
+++ b/src/components/RequestSamples/RequestSamples.tsx
@@ -4,7 +4,7 @@ import { OperationModel, RedocNormalizedOptions } from '../../services';
 import { PayloadSamples } from '../PayloadSamples/PayloadSamples';
 import { SourceCodeWithCopy } from '../SourceCode/SourceCode';
 
-import { RightPanelHeader, Tab, TabList, TabPanel, Tabs } from '../../common-elements';
+import { Tab, TabList, TabPanel, Tabs } from '../../common-elements';
 import { OptionsContext } from '../OptionsProvider';
 
 export interface RequestSamplesProps {
@@ -24,15 +24,10 @@ export class RequestSamples extends React.Component<RequestSamplesProps> {
     const samples = operation.codeSamples;
 
     const hasSamples = hasBodySample || samples.length > 0;
-    const hideTabList =
-      samples.length + (hasBodySample ? 1 : 0) === 1
-        ? this.context.hideSingleRequestSampleTab
-        : false;
+    const hideTabList = true;
     return (
       (hasSamples && (
         <div>
-          <RightPanelHeader> Request samples </RightPanelHeader>
-
           <Tabs defaultIndex={0}>
             <TabList hidden={hideTabList}>
               {hasBodySample && <Tab key="payload"> Payload </Tab>}
diff --git a/src/components/ResponseSamples/ResponseSamples.tsx b/src/components/ResponseSamples/ResponseSamples.tsx
index 3f9eefa0..0fb6c4d8 100644
--- a/src/components/ResponseSamples/ResponseSamples.tsx
+++ b/src/components/ResponseSamples/ResponseSamples.tsx
@@ -3,7 +3,7 @@ import * as React from 'react';
 
 import { OperationModel } from '../../services/models';
 
-import { RightPanelHeader, Tab, TabList, TabPanel, Tabs } from '../../common-elements';
+import { Tab, TabList, TabPanel, Tabs } from '../../common-elements';
 import { PayloadSamples } from '../PayloadSamples/PayloadSamples';
 
 export interface ResponseSamplesProps {
@@ -23,8 +23,6 @@ export class ResponseSamples extends React.Component<ResponseSamplesProps> {
     return (
       (responses.length > 0 && (
         <div>
-          <RightPanelHeader> Response samples </RightPanelHeader>
-
           <Tabs defaultIndex={0}>
             <TabList>
               {responses.map(response => (
diff --git a/src/components/Responses/Response.tsx b/src/components/Responses/Response.tsx
index 7c46a659..164ee62a 100644
--- a/src/components/Responses/Response.tsx
+++ b/src/components/Responses/Response.tsx
@@ -3,7 +3,7 @@ import * as React from 'react';
 
 import { ResponseModel } from '../../services/models';
 import { ResponseDetails } from './ResponseDetails';
-import { ResponseDetailsWrap, StyledResponseTitle } from './styled.elements';
+import { ResponseDetailsWrap } from './styled.elements';
 
 @observer
 export class ResponseView extends React.Component<{ response: ResponseModel }> {
@@ -12,7 +12,7 @@ export class ResponseView extends React.Component<{ response: ResponseModel }> {
   };
 
   render() {
-    const { headers, type, summary, description, code, expanded, content } = this.props.response;
+    const { headers, description, expanded, content } = this.props.response;
     const mimes =
       content === undefined ? [] : content.mediaTypes.filter(mime => mime.schema !== undefined);
 
@@ -20,14 +20,6 @@ export class ResponseView extends React.Component<{ response: ResponseModel }> {
 
     return (
       <div>
-        <StyledResponseTitle
-          onClick={this.toggle}
-          type={type}
-          empty={empty}
-          title={summary || ''}
-          code={code}
-          opened={expanded}
-        />
         {expanded &&
           !empty && (
             <ResponseDetailsWrap>
diff --git a/src/components/Responses/ResponseTitle.tsx b/src/components/Responses/ResponseTitle.tsx
index be66fdc8..0e50c7a0 100644
--- a/src/components/Responses/ResponseTitle.tsx
+++ b/src/components/Responses/ResponseTitle.tsx
@@ -1,6 +1,5 @@
 import * as React from 'react';
 
-import { ShelfIcon } from '../../common-elements';
 import { Markdown } from '../Markdown/Markdown';
 
 export interface ResponseTitleProps {
@@ -15,17 +14,9 @@ export interface ResponseTitleProps {
 
 export class ResponseTitle extends React.PureComponent<ResponseTitleProps> {
   render() {
-    const { title, type, empty, code, opened, className, onClick } = this.props;
+    const { title, empty, code, className, onClick } = this.props;
     return (
       <div className={className} onClick={(!empty && onClick) || undefined}>
-        {!empty && (
-          <ShelfIcon
-            size={'1.5em'}
-            color={type}
-            direction={opened ? 'up' : 'down'}
-            float={'left'}
-          />
-        )}
         <strong>{code} </strong>
         <Markdown compact={true} inline={true} source={title} />
       </div>
diff --git a/src/components/Responses/ResponsesList.tsx b/src/components/Responses/ResponsesList.tsx
index da465489..eee2b6ed 100644
--- a/src/components/Responses/ResponsesList.tsx
+++ b/src/components/Responses/ResponsesList.tsx
@@ -1,16 +1,7 @@
 import * as React from 'react';
 import { ResponseModel } from '../../services/models';
-import styled from '../../styled-components';
 import { ResponseView } from './Response';
 
-const ResponsesHeader = styled.h3`
-  font-size: 18px;
-  padding: 0.2em 0;
-  margin: 3em 0 1.1em;
-  color: #253137;
-  font-weight: normal;
-`;
-
 export interface ResponseListProps {
   responses: ResponseModel[];
 }
@@ -25,7 +16,6 @@ export class ResponsesList extends React.PureComponent<ResponseListProps> {
 
     return (
       <div>
-        <ResponsesHeader> Responses </ResponsesHeader>
         {responses.map(response => {
           return <ResponseView key={response.code} response={response} />;
         })}
diff --git a/src/services/models/Response.ts b/src/services/models/Response.ts
index 3fc4d06c..863ef994 100644
--- a/src/services/models/Response.ts
+++ b/src/services/models/Response.ts
@@ -26,7 +26,7 @@ export class ResponseModel {
     infoOrRef: Referenced<OpenAPIResponse>,
     options: RedocNormalizedOptions,
   ) {
-    this.expanded = options.expandResponses === 'all' || options.expandResponses[code];
+    this.expanded = true;
 
     const info = parser.deref(infoOrRef);
     parser.exitRef(infoOrRef);
diff --git a/src/theme.ts b/src/theme.ts
index c9cda220..8ae57f6b 100644
--- a/src/theme.ts
+++ b/src/theme.ts
@@ -4,7 +4,7 @@ const defaultTheme: ThemeInterface = {
   spacing: {
     unit: 5,
     sectionHorizontal: ({ spacing }) => spacing.unit * 8,
-    sectionVertical: ({ spacing }) => spacing.unit * 8,
+    sectionVertical: ({ spacing }) => spacing.unit * 20,
   },
   breakpoints: {
     small: '50rem',
@@ -81,7 +81,7 @@ const defaultTheme: ThemeInterface = {
     typeNameColor: theme => theme.colors.text.secondary,
     typeTitleColor: theme => theme.schema.typeNameColor,
     requireLabelColor: theme => theme.colors.error.main,
-    labelsTextSize: '0.9em',
+    labelsTextSize: '0.7em',
     nestingSpacing: '1em',
     nestedBackground: '#ffffff',
     arrow: {
@@ -90,7 +90,7 @@ const defaultTheme: ThemeInterface = {
     },
   },
   typography: {
-    fontSize: '16px',
+    fontSize: '14px',
     lineHeight: '1.6em',
     fontWeightRegular: '400',
     fontWeightBold: '600',
@@ -144,7 +144,7 @@ const defaultTheme: ThemeInterface = {
     textColor: '#ffffff',
   },
   codeSample: {
-    backgroundColor: ({ rightPanel }) => darken(0.1, rightPanel.backgroundColor),
+    backgroundColor: '#4f566b',
   },
 };