Using the concepts that I learnt in previous projects and completing this project faster.
Latest solutions
Mobile first using CSS flexbox
#bem#sass/scssSubmitted over 3 years agoWould appreciate feedback on use of SCSS.
Latest comments
- @TheAashayWhat are you most proud of, and what would you do differently next time?@SJ-74
Well done on your challenge. Regarding your CSS. I'd like to point out a couple of things.
- Using max-height should be avoided as much as you can.
- px can be used to replace shadow length and padding can be set using ems.
- @DuloeW@SJ-74
Great job on completing the challenge. Few things I want to point out:
- The background color of body is different compared to the design. (Color codes are provided in style-guide.md file).
- Giving some shadow to the card.
- Padding inside the card is more compared to the design.
- Instead of the generic
div
tag, try using semantic html tags. - Avoid giving a fixed height and width. You should use
max-width
andcalc
for such operations. - Should use relative units instead of px for size.
- @moessam322@SJ-74
Greetings,
I'd suggest a few updates/changes.
- Your
.project
is wrapping all the content inside. So, instead of giving it a 'width:1440px` you should have left it because on screen-size larger than 1440 the background-color kinda changes at the ends (changes to white). - For some reason the heading text (your h1) is not visible on my screen.
- Also, I'd suggest going mobile first flow(though it's totally up to you) because you don't have to write as much media query. It reduces most of your work. So, instead of changing everything, you just have to change a few things.
Marked as helpful - Your