Submitted 16 days ago

Meet landing page SCSS Grid/Flex

#sass/scss

Matthef•170

@MatFrat95

Preview site View code

This is a solution for...Meet landing page

HTML
CSS

1newbie

Design comparison

SolutionDesign

Solution retrospective

What are you most proud of, and what would you do differently next time?

This was very problematic project, from the very beginning I faced multiple problems with particular elements, mixins, responsiveness and much more. The most important fact is, I solved all of those problems and learn a lot from my own mistakes.

Besides that, I know also what should I improve to write better code, I will implement knowdledge i obtain here in upcomming projects to create code easier to read and edid and more reusable.

I also implemented mixins with variables and used scss to divide code between separate files, thats very handy. Ill try to use more varibles other than colors in next projects and clamp() as font size unit.

What challenges did you encounter, and how did you overcome them?

I had problems with:

using mixins to with variables included, it was hard for me tu understand why my code was not overwriteable for @media (found out I had to put images to separate section)
correct order of files in workspace which cause problem with emulating site via github (trial and error)
responsivenes of site obtained by flex and grid used as mixins I'm sure there was a lot of other problems but those took me the most time to fix on my own.

What specific areas of your project would you like help with?

Can i use 'margin: -3rem;' to move element up? And .. why I can't see picture from footer here and I can see it on live server? Besides that I will appreciate if anyone give me any feedback :D

Community feedback

P
Kumani Kidd•150
@amancalledkiddPosted 11 days ago
Great work - the landing page looks great, matches the designs and is completely responsive!

Good use of semantic html. I would recommend using the <header> and <footer> tags instead of a section or div as this helps screen readers.

Code is well structured, well done with the mixins and seperations of scss. Also with the comments it is very readable.

negative margins can be useful to get content in the right place but can lead to issues on different sizes screens

In regards to the footer img, this is happening because (line 44 inside _media.scss) background-image: url(../../starter-code/assets/desktop/image-footer.jpg) the url is incorrect as you have an extra ../. What you have written is correct if using _media.scss directly. But because the css is being called inside style.scss which is a level up it does not need the extra ../.

Again great work, hope this feeback helps!
0

Please log in to post a comment

Join our Discord community

Join thousands of Frontend Mentor community members taking the challenges, sharing resources, helping each other, and chatting about all things front-end!

Join our Discord