-
Notifications
You must be signed in to change notification settings - Fork 100
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
158 remove colors file #260
base: staging
Are you sure you want to change the base?
158 remove colors file #260
Conversation
…ommunity#235) * test: add e2e infra with playwright and a sample test * test: add github action for e2e test * test: add config for framework spin-up
* feat: add Yedidya Rashi profile (Pull-Request-Community#220) * fix: randomShuffle isn't working correctly (Pull-Request-Community#167)
…issue241 Yedidyar/issue241
* inital commit * revert change Co-authored-by: noa <[email protected]>
Preview on netlify: https://deploy-preview-260--pull-request.netlify.app/ |
✅ Deploy Preview for agitated-hoover-61371e ready!Built without sensitive environment variables
To edit notification comments on pull requests, go to your Netlify site settings. |
4e8f3c0
to
28ac821
Compare
6a7c4da
to
449fdf3
Compare
@Noamili please fix conflicts |
9fd6b96
to
d97a360
Compare
rgba(220, 102, 94, 0.6) 100% | ||
); | ||
$purple-100: rgba(11, 0, 142, 0.3); | ||
$purple-200: rgba(11, 0, 142, 0.1); |
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.
how can I avoid using opacity?
@@ -14,6 +16,6 @@ | |||
align-items: center; | |||
justify-content: center; | |||
font-size: 14px; |
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.
should I change this to rem?
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.
Yes please :)
@@ -10,7 +12,7 @@ | |||
justify-content: center; | |||
font-weight: bold; | |||
font-size: 36px; |
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.
should I change this to rem?
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.
Yes please :)
close #158