@EL132
@davidFreelance19
Submitted
It is an excellent challenge to test your skills with position, display, custom properties, etc. I'm happy with the final result and little by little I'm improving. I would like to receive new comments towards my code
@EL132
Posted
Hey David!
Awesome solution that is extremely well done even to the small details! A few small things that will be helpful in the long run:
I hope this helps! If it does, please let me know to help my commenting improve! Elias
Marked as helpful
@Cruc1f1x
Submitted
Feedback Welcomed
@EL132
Posted
Hello!
Your solution looks great!
I have two main pieces of advice:
Try to avoid using an id for every element. Instead, try to use a class. Using an id for every element takes away the power that comes with having an id on an element. Since an id is an extremely specific classification, you should only use sparingly. A class selector will do the same thing while leaving you the ability to specify further on an element using id. This all relates to the specificity of selectors in CSS and HTML (see link here: https://www.w3schools.com/css/css_specificity.asp )
Try to use semantic components in your html instead of using a div for every element. Using semantics allows for people reading your code to better understand what each element is relating to. For example, instead of using a <div> Title </div>, you can use <h1> Title </h1> to let your code reader know that this element is a title.
I hope this helps! Elias
Marked as helpful
@ribasgil
Submitted
Hi, this is my second project all by myself. I've been studying html/css/js alone for 2 weeks now. Today I realized that I still have a long way to go, even looking at code from other devs I was not able to make mobile responsiveness work at all. It's going to be a rough road, but I won't stop now. When I have enough knowledge, I'll come back to finish the project, thank you.
@EL132
Posted
Hello!
I'm happy to see you got this far on the project! I know you said you would come back to this project, but I figured I would leave a comment to help you get there sooner/easier (hopefully). I recently also learned responsive design and successfully implemented it using @media queries. A media query is a way that lets your webpage respond to certain conditions. When I am designing for my mobile designs, I use the following syntax:
@media (max-width: 1380px) { .wrapper { width: 400px; } }
This is an example of my page needing to do something different when the width of the page goes below 1380 pixels. You can think of it like "Once my page goes below 1380px in width, apply the properties inside the curly braces {}" Hopefully, this example is enough to get you started; this technique can be used for a lot of topics and is a great place to start.
An alternative method is a CSS library called Bootstrap; personally, I do not like Bootstrap, but many do like if because it makes resizing websites very easy. Here is a link to the bootstrap documentation for its responsive grid design: https://getbootstrap.com/docs/4.0/layout/grid/
I hope this helps! Elias
Marked as helpful
@Mohamed33362
Submitted
it was difficult for me , i know i have a loot of wrong codes or tags but i want a helpful feedback to improve my skills in css and javascript spacific
@EL132
Posted
Hey Mohamed!
Nice job completing the design for the project! I would love to help you with your JavaScript section of the code, but I don't see your code when I click on the view code link. Could you put a link in a response to this comment that would take me to your code? Thank you!
Elias
