Looking to hire developers?
@Median21
@Marianellag1
Submitted
Hello! From previous comments, Its truly-true that if you use bootstrap and framework, "helpful sites" for challenges, its not a good thing because you don't learn the raw css that is much more useful. And cannot agree more. Still learning tho, so any help to improve in any part of the code will be appreciated. Thank you!
@Median21
Posted
Hello Mgamboa 👋
I have few concerns regarding the design:
I suggest adding more padding in your column or first class, so that it has a bigger spaces on the sides.
Next, is the color of the image, I suggest researching about the mix-blend-mode property, to be able to match the color of the image in the design. Specifically for me, I used the mix-blend-mode: multiply
You can check it out here: w3schools mix-blend-mode
Overall, you did a very good job. Keep it up! 💪
Marked as helpful
@MelvinAguilar
Submitted
Hi there 👋, I’m Melvin and this is my solution for this challenge. 🚀
Help:
Built With:
Any suggestions on how I can improve and reduce unnecessary code are welcome!
Thank you. 😊✌️
@Median21
Posted
Wowww, you really did a good job in doing the challenge!
Your design is spot on and the extra hover effects are cool!
I don't really know what to say, because I am not that experienced yet, but amazing job overall!🙏🙏🙏
Marked as helpful
@Cyclone3603
Submitted
Hey there! This is my first project on HTML and CSS after a LOOOOOONG time of not coding at all. This was a nice, simple site to help me get into the swing of things, and I feel like I really needed that.
Please let me know about any improvements I can make in the comments!
@Median21
Posted
Hello there! You did a good job following the designs and completing the challenge 😀
Here are the problems I would like to address:
- Don't start your code by putting it in a <div> even though it is still acceptable, since this challenge only requires one content, it is still a good practice to use semantic elements like <header>, <main>, and many more. So that it is easier to be seen in search engines and makes it more specific.
- The <br> tag is not a semantic element. If a screen reader user is reading the page, they will hear "line break", which breaks the flow of the content. Instead, use CSS properties like margin and padding to add vertical space between elements. (I got this tip from a feedback, so I also wanted it to share to you)
Overall, you did a very good job! Knowing that you just came back from coding 💪
@HarshanaLakmal98
Submitted
@Median21
Posted
The size of your QR is too big, I suggest reducing the width. But overall, you were able to copy the main design. Well done!
Marked as helpful
