@visualdenniss
Posted
Hello there,
great job completing this successfully! Few suggestions for further improvements:
- Add a little bit more gap between 'Share' and button icons.
- No need to choose headers based on their size, e.g. h5 to get a smaller. Headings on a page should follow an order, h1 , h2 etc. and use CSS to change their style.
- Consider using media queries to add responsiveness for mobile view.
- Variable naming in JS looks a little confusing, as one would except 'buttons' would be multiple buttons, but you are just selecting one button, so better to name as 'button' etc.
Hope these recommendations are helpful!