Skip to content
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

Add Feature/swipe component #36

Open
wants to merge 19 commits into
base: main
Choose a base branch
from
Open

Add Feature/swipe component #36

wants to merge 19 commits into from

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.

5 participants