-
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 2 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; | ||
|
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 |
---|---|---|
@@ -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.