Hey folks, I have completed the challenge of Social Media Dashboard with Theme Switcher using React JS and SASS. Please provide your feedback regarding my work. Your every comment is valuable and really appreciated. Thank you. Love PS.
Lazar Nikolov
@nikolovlazarAll comments
- @parthee11Submitted almost 4 years ago@nikolovlazarPosted almost 4 years ago
Wow this is gorgeous! Superb! Love it! š
The only thing left is to fix the Tablet breakpoint. Try to get creative with the layout. Usually if there are 4 columns on desktop, there should be 2 on tablet and 1 on mobile. Try changing your grid columns to 2 and see what happens.
Keep up the great work yo! š
2 - @irfan-fauziSubmitted almost 4 years ago
feedback please my friends..
@nikolovlazarPosted almost 4 years agoHi Irfan! š
Your solution looks really good! I noticed the navbar items are not capitalized. You can achieve that by setting
text-transform: uppercase;
to them. Also, keep an eye on the color. I noticed some of the text you have in your solution is painted with a different color than the design. The colors might look the same, but it affects the overall design of the website. You can use a simple Color Picker to discover even the slightest color differences. As for the "Download extension" section, try addingmargin-top
to the elements to achieve that effect.All in all, good job! Keep up the great work! š
1 - @enigmireSubmitted almost 4 years ago
Thanks for the comments as always guys.
However for this challenge, I have problem changing the color of the image for the footer to fit the design, couldn't get the white color. I hope to get that now.
Also, I don't know why the
<p>
at the hero session is bold. Have checked the code over and over and all seems fine to me.Kindly help.
@nikolovlazarPosted almost 4 years agoIyanu this is just perfect! Everything's nicely executed, responsive and by the design. Great great great job! š
As for the
<p>
tag font, I notice that you're using the "Poppins" font, which is only imported at 600 weight. Instead, you should be using the "Open Sans" font, because it has the 400 weight you need. Check out thestyle-guide.md
file under "Body, Call-to-actions". That piece of content is not a heading, so you should use the styling for "Body, Call-to-actions".This is so amazing! Keep up the great work Iyanu! š
1 - @EsraaGamal-22Submitted almost 4 years ago
rate my design please, Any feedback and suggestions on how I can improve are very welcome!
@nikolovlazarPosted almost 4 years agoGreat job Esraa! š
I could suggest you wrap your content in a "container" div to keep everything tidy. Also, pay attention to the small details, your background is different, you're missing the shadows, and the spacing between the cards is not consistent. Noticing these details will become easier and easier as you code, so don't worry, you're on the right path! For reference, you can research the "Masonry Grid" layout and its responsiveness. It'll give you an idea on how to refactor your solution and make it "top shelf".
This is awesome Esraa! Keep up the great work!
0 - @zky63Submitted almost 4 years ago
Please let me know if you see any areas of improvement within the code. I messed up by using grid-auto-fill: row so I had to use flexbox to reorder the elements for the mobile version. Will try to fix that some other time.
Thanks!
@nikolovlazarPosted almost 4 years agoGreat job Zach!
The layout looks pretty much pixel perfect! I can notice that you're missing the card shadows. That would be very easy to fix for you, since you're a Pro user and you have the Figma design file.
On the other hand, the responsiveness is not perfect. There's a point between the Desktop breakpoint and the Mobile breakpoint where the content is bleeding off the edge. This layout is not simple though, it's a complex layout. What you can do is research the Masonry Grid layout and its responsiveness. It'll give you some ideas on how to refactor your solution and make it responsive.
Keep up the great work man! Cheers š
2 - @FaridJuniorSubmitted almost 4 years ago
hello it is my first try to use sass If there is any feedback I will appreciate it
@nikolovlazarPosted almost 4 years agoWow this is awesome! š
Great job Mohamed! By the way, I'm looking at your solution from a 27" iMac, and I can suggest you wrap your whole page in a "container", basically set the max width of the website to be 1440px for example. Right now the contents are spread from edge to edge, and there's a lot of spacing between the 4 features below the hero. Wrapping your content into a "container" div will fix that issue š
Keep up the great work man! This is amazing!
2 - @cd1414Submitted almost 4 years ago
I am trying to learn something new and I would like to here any feedback, thanks
@nikolovlazarPosted almost 4 years agoLooking good! You could also center the FAQ section vertically and it'll be just like the design! š Great job Christian!
1