@iliAsghar
Posted
great job! πͺ you did well in layout and Positioning the elements. some things that I think needs a bit of a tweak:
- i noticed you used <section> for designing the buttons . it's usually recommended to use
<a></a>
for buttons (other than form buttons . those should be<button>
, because it can submit a form entry) . it's because the<button>
tag or any other tag can't link to another website . according to MDN - <Section>, <section> is a generic sectioning element, and should only be used if there isn't a more specific element to represent it. - as it was intended ( i think it was, maybe i'm mistaken πΆ) , it's better to have a
:focus
state for the buttons . simply an outline around them to help the keyboard users. - even if a project is small and can be done without them , Defining a class for each element will get you used to naming and nesting the elements better . one simple naming convention is the BEM naming . you can check it out HERE.
i hope you have fun continuing your Web Development journey , Keep up the good work! β π€π
@iliAsghar
Posted
@iliAsghar i just noticed you have a bunch of solutions in your profile ! im more in a learning journey myself, and since the solution was recommended for me , i thought i'd share some of my exp here . πΆπ
@mussino
Posted
Is OK! thanks for your feedback, as I said is always welcome and you were right I should use all those suggestion you told me. Buttons are welcome! @iliAsghar