Skip to content

Conversation

@svhoy
Copy link
Member

@svhoy svhoy commented Feb 25, 2021

#16

@svhoy svhoy requested a review from oezguerisbert February 25, 2021 22:38
});
</script>

<style lang='scss'>
Copy link
Contributor

Choose a reason for hiding this comment

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

Move all the design part into the src/assets/scss/base/main.scss

Copy link
Member Author

Choose a reason for hiding this comment

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

So we only use a one scss file?

border-radius: 3rem;
color: $textColor;
padding: 2rem;
width:300px;
Copy link
Contributor

Choose a reason for hiding this comment

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

Please change to rem (1rem = 10px). Here it would be width: 30rem

Copy link
Contributor

@oezguerisbert oezguerisbert left a comment

Choose a reason for hiding this comment

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

Check comments :)

oezguerisbert and others added 4 commits February 28, 2021 22:51
* fix: Fixed styling for Input, Radio, Checkbox, Texarea. Added Global Styling into vue.config.js + inputs.scss

* fix: scss setup correction

* style: remove shared styles from components

* style: reformat code

* style: px to rem

* refactor: rem instead of px, font-size fixed

* refactor: Refactored design and made it BEM-coding-styled.

* refactor: Refactored design and made shortened BEM via SCSS.

Co-authored-by: Robin Kemna <[email protected]>
@svhoy svhoy requested a review from oezguerisbert March 5, 2021 19:40
@oezguerisbert oezguerisbert requested a review from RobinKemna March 5, 2021 19:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants