Skip to content

Commit

Permalink
Update create connection page to full width, and fix trivial UI issues (
Browse files Browse the repository at this point in the history
#21352)

* Update big prop in FormPageContent to expand to full width

* Fix Normalization & Tranformation heading size

* Update FormPageContent to scss and add bottom padding

* Remove max-width from ConnectionReplicationPage
  • Loading branch information
edmundito authored and jbfbell committed Jan 13, 2023
1 parent 218b42a commit 2256412
Show file tree
Hide file tree
Showing 4 changed files with 23 additions and 9 deletions.
Original file line number Diff line number Diff line change
@@ -0,0 +1,11 @@
@use "scss/variables";

.container {
margin: 13px auto 0;
padding-bottom: variables.$spacing-page-bottom;

&:not(.big) {
width: 80%;
max-width: 813px;
}
}
17 changes: 11 additions & 6 deletions airbyte-webapp/src/components/ConnectorBlocks/FormPageContent.tsx
Original file line number Diff line number Diff line change
@@ -1,9 +1,14 @@
import styled from "styled-components";
import classNames from "classnames";
import { PropsWithChildren } from "react";

const FormPageContent = styled.div<{ big?: boolean }>`
width: 80%;
max-width: ${({ big }) => (big ? 1279 : 813)}px;
margin: 13px auto;
`;
import styles from "./FormPageContent.module.scss";

interface FormPageContentProps {
big?: boolean;
}

const FormPageContent: React.FC<PropsWithChildren<FormPageContentProps>> = ({ big, children }) => (
<div className={classNames(styles.container, { [styles.big]: big })}>{children}</div>
);

export default FormPageContent;
Original file line number Diff line number Diff line change
@@ -1,8 +1,6 @@
@use "scss/variables";

.content {
max-width: calc(100% - 2 * (#{variables.$spacing-xl}));
margin: 0 variables.$spacing-xl;
padding-bottom: variables.$spacing-md;

> form {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -38,7 +38,7 @@ export const OperationsSection: React.FC<OperationsSectionProps> = ({
<Card>
<Section>
{supportsNormalization || supportsTransformations ? (
<Heading as="h5">
<Heading as="h2" size="sm">
{[
supportsNormalization && formatMessage({ id: "connectionForm.normalization.title" }),
supportsTransformations && formatMessage({ id: "connectionForm.transformation.title" }),
Expand Down

0 comments on commit 2256412

Please sign in to comment.