Skip to content
OverviewSolutions
11
Comments
6

Abdalla Rahmah

@AbdallaRahmahUnited Arab Emirates260 points

Hello there 👋 , I'm a front-end developer with entry-level experience specializing in web development, HTML, CSS, Javascript, Typescript, Bootstrap, and React. Adept at identifying opportunities to enhance front-end design and improve the user experience.

I’m currently learning...

web development - React - Typescript - Javascript - CSS - Bootstrap - HTML.

Latest solutions

View more solutions

Latest comments

  • Abdalla Rahmah•260
    @AbdallaRahmahPosted over 2 years ago

    Hey Kelpy, congratulations on completing the challenge! You did a great job 😉

    Let me give you some little tips for optimizing your code:

    • use the main tag insted of the div tag.
    • add a transition on your elements that have hover action to make smooth.
    Marked as helpful
    1
  • Abdalla Rahmah•260
    @AbdallaRahmahPosted over 2 years ago

    nice work Nell, i like your work and i have some ideas for you:

    • instead of adding a large padding to the body element you can add min-height: 100vh and justify content: center this will help you center the main element.

    • also you can remove the article-container div and duplicate its styles to the main tag.

    best of luck for you.

    0
  • Abdalla Rahmah•260
    @AbdallaRahmahPosted almost 3 years ago

    nice work Shann you can lern about css flexbox to center the card and give the card a flexible width value that will make it responsive.

    1
  • Abdalla Rahmah•260
    @AbdallaRahmahPosted about 3 years ago

    nice work

    1
  • Abdalla Rahmah•260
    @AbdallaRahmahPosted about 3 years ago

    Hi @brunafelix79, nice work but you can use semantic elements to improve your SEO.

    Marked as helpful
    0
  • Abdalla Rahmah•260
    @AbdallaRahmahPosted about 3 years ago

    nice work Dinah, i recommend you to use flexbox insted of position to center elements, also you use the box-shadow property to add nice look to the box.

    0