Skip to content
This repository has been archived by the owner on Nov 30, 2022. It is now read-only.

Removes some magic numbers used for resizing of columns. #277

Merged
merged 3 commits into from Jun 27, 2019

Conversation

hrigner
Copy link

@hrigner hrigner commented Jun 27, 2019

Also added css custom properties to index.pcss for some css being used all over catwalk.

@netlify
Copy link

netlify bot commented Jun 27, 2019

Deploy preview for catwalk-qlikcore ready!

Built with commit d70ca8a

https://deploy-preview-277--catwalk-qlikcore.netlify.com

@@ -4,7 +4,7 @@
}

.app {
background-color: #f2f2f2;
background-color: var(--light-grey);
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Still pretty specific (...grey) - maybe something like primary/secondary?
If someone whats to style it differently later

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Using primary-color and primary-color-dark!

Copy link
Member

@axelssonHakan axelssonHakan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice 👍

@hrigner hrigner merged commit af6f61f into master Jun 27, 2019
@axelssonHakan axelssonHakan deleted the css-custom-props branch July 1, 2019 10:08
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants