Skip to content
OverviewSolutions
45
Comments
627

Kamasah-Dickson

@Kamasah-DicksonAccra/Ghana5,570 points

As a frontend developer, I like spending most of my time coding. It has been a very great journey and experience since I decided to become a frontend developer and I think this is one of the best decisions I've made 💻🚀

I’m currently learning...

NodeJS/express

Latest solutions

View more solutions

Latest comments

  • Kamasah-Dickson•5,570
    @Kamasah-Dickson

    Hey @sgrazys i checked your solution. it looks great and responsive. but on mobile devices the cards looks a bit small and there is so much white spaces on the left and right. try to increase the max-width of the cards. by the way great job.👏👏

  • Kamasah-Dickson•5,570
    @Kamasah-Dickson

    @scott Tabor you did I really great job. your solution was very responsive as well. But one of the things you forgot to add was the type attribute for your button. some of the importance are.

    1. It helps to define the role and behavior of the button for users and assistive technologies.
    2. Providing a type for your buttons also helps improve the accessibility of your website.
    3. The type also determines if the button is a submit,reset or just a button.

    I hope this was helpful. Happy coding🔥

    Marked as helpful
  • Kamasah-Dickson•5,570
    @Kamasah-Dickson

    Am finding it hard to fix the accessibility issues with react...should I run npm run dev or what. any idea??

  • Kamasah-Dickson•5,570
    @Kamasah-Dickson

    Your solution looks great and to help you with the accessibility issue 1.There should always be an h1 in your projects. It should be the first head tag you use. And should reduce in descending order. h1....h6

    Happy coding🎉

  • Kamasah-Dickson•5,570
    @Kamasah-Dickson

    Your solution looks great but i want to help you with some small tips.

    1 . Do not specify any height in the grid items. This sometimes breaks layout and might stress you some other time when things are not working as expected. Instead allow the items to determine their own height. What i mean is that paddings and margins applied to the grid items will automatically determine the heights of the grid items. I hope you get me.

    2 . To help you with the accessibility issues, remove the attribution class and change the div to a footer

    Do not use a section tag if there won't be any direct head tag I mean h1....h6 always use a section tag if there would be a directed head tag in your page otherwise use a div instead.

    Good job there👍🎉 Happy coding

    Marked as helpful
  • Kamasah-Dickson•5,570
    @Kamasah-Dickson

    hi there🖐 your solution looks very great and responsive but one thing is that the background-color of the feature section is not set. it should be different from the actual body background-color which I know you noticed too. But also one thing again is that the svg is just above the feature section...kinda before. it just blends with the background-color of the feature section. besides, Great job there👍

    Happy coding🔥🔥

View more comments