-
Notifications
You must be signed in to change notification settings - Fork 4.6k
🪟 🎉 New theme for Airbyte UI #18817
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
🪟 🎉 New theme for Airbyte UI #18817
Changes from 6 commits
4a514ce
625832a
b87e63f
6353226
8fe387a
481f2bc
34424c1
c82df6b
72ca81d
6171a08
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
This file was deleted.
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -8,7 +8,7 @@ interface Props { | |
} | ||
|
||
const Box = styled.div` | ||
background: ${({ theme }) => theme.darkBeigeColor}; | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. |
||
background: ${({ theme }) => theme.yellow100}; | ||
border-radius: 8px; | ||
padding: 18px 25px 22px; | ||
font-size: 14px; | ||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -20,7 +20,7 @@ const StepView = styled.div<{ | |
min-width: ${({ lightMode }) => (lightMode ? "100px" : "auto")}; | ||
min-height: 28px; | ||
padding: 6px 14px; | ||
border-radius: 4px; | ||
border-radius: 28px; | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. |
||
pointer-events: ${({ isActive, nonClickable }) => (isActive || nonClickable ? "none" : "all")}; | ||
cursor: ${({ nonClickable }) => (nonClickable ? "default" : "pointer")}; | ||
text-align: center; | ||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -84,7 +84,7 @@ const Th = styled.th<TableHeaderProps>` | |
font-size: ${({ light }) => (light ? 11 : 10)}px; | ||
line-height: 12px; | ||
color: ${({ theme, highlighted }) => (highlighted ? theme.whiteColor : theme.lightTextColor)}; | ||
border-bottom: ${({ theme, light }) => (light ? "none" : ` 1px solid ${theme.backgroundColor}`)}; | ||
border-bottom: none; | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. ℹ️ This border was anyway barely visible (and not sure why we at all tried to have a beige border below the dark blue table header), so I removed it. |
||
width: ${({ collapse, customWidth }) => (customWidth ? `${customWidth}%` : collapse ? "0.0000000001%" : "auto")}; | ||
font-weight: ${({ light }) => (light ? 400 : 600)}; | ||
text-transform: ${({ light }) => (light ? "capitalize" : "uppercase")}; | ||
|
This file was deleted.
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -7,7 +7,7 @@ | |
} | ||
|
||
.connectionTitle { | ||
color: colors.$orange; | ||
color: colors.$blue-500; | ||
} | ||
|
||
.connectionDeleted { | ||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -27,7 +27,7 @@ const run = keyframes` | |
const Bar = styled.div` | ||
width: 100%; | ||
height: 49px; | ||
background: ${({ theme }) => theme.darkBeigeColor} url("/rectangle.svg"); | ||
background: #ffebd7 url("/rectangle.svg"); | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. ℹ️ Last place this beige is still used, but since we're planning to remove the onboarding page altogether soon, I just inlined the color for now here, instead of redesigning this progress bar. |
||
color: ${({ theme }) => theme.redColor}; | ||
border-radius: 15px; | ||
font-weight: 500; | ||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,4 +1,6 @@ | ||
/* stylelint-disable color-no-hex, color-hex-length */ | ||
$blue-30: #f4f4ff; | ||
$blue-40: #f0efff; | ||
Comment on lines
+2
to
+3
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. ℹ️ Took from our color palette in Figma, those where still missing here. |
||
$blue-50: #eae9ff; | ||
$blue-100: #cbc8ff; | ||
$blue-200: #a6a4ff; | ||
|
@@ -74,10 +76,6 @@ $red-800: #bc0042; | |
$red-900: #99003f; | ||
$red: $red-300; | ||
|
||
$beige-50: #fef9f4; | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. ℹ️ No more beige color!!! |
||
$beige-100: #ffebd7; | ||
$beige: $beige-50; | ||
|
||
$black: #000000; | ||
$white: #ffffff; | ||
|
||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,25 @@ | ||
@use "./colors"; | ||
@use "./fonts"; | ||
|
||
html, | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. ℹ️ In the styled-component variant this still had a |
||
body, | ||
#root { | ||
height: 100%; | ||
width: 100%; | ||
padding: 0; | ||
margin: 0; | ||
font-weight: normal; | ||
-webkit-font-smoothing: antialiased; | ||
color: colors.$dark-blue; | ||
font-family: fonts.$primary; | ||
background: colors.$grey-50; | ||
font-size: 14px; | ||
} | ||
|
||
button, input, textarea { | ||
font-family: fonts.$primary; | ||
} | ||
|
||
* { | ||
box-sizing: border-box; | ||
} |
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -16,8 +16,6 @@ | |
margin-bottom: 24px; | ||
} | ||
|
||
background: radial-gradient(35.57% 35.57% at 50% 50%, colors.$white 0%, colors.$white 55.87%, colors.$beige 100%); | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. ℹ️ On the new background it looks nicer just not having any radial gradient behind the error view. |
||
|
||
.content { | ||
max-width: 600px; | ||
text-align: center; | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ℹ️ Removing the old GlobalStyle component using styled-components and replacing it by a
global.scss
file that's loaded in index.tsx.