bouaik / embedding-images-and-video

Hey, there! 👋

I'm a highly motivated and dynamic Full Stack Web Developer from Morocco 🇲🇦.. I am passionate about Web Development and open source. 🤗 I’m looking forward to collaborating on any Open Source projects, so please feel free to contact me.

The power of Open Source is the power of the people. The people rule.

-- Philippe Kahn

I am currently working with 👇

embedding-images-and-video's Issues

Peer to peer Code review

Good work 👏 ! You can use font awesome icons instead of images and I think you should add UpNext and autoplay toggle on off icons also in your side bar !

main tag instead of section

it will be better to use the semantic tag of main instead of section. also instead of a div with side-bar id, you can use the aside semantic tag. the last thing you should try with your partner to center the navbar and also leave spaces between the icon and the search bar.

Image tags and Branch names

• Include alt on image tags. Just in case the image doesn't load, the alt will describe what the image is.
• Use descriptive branch names.