Abdullah

Latest comments

• 

  Wojtek Natkaniec•170

  @wojocode

  Submitted almost 3 years ago

  

  Feature-Cards

  2

  20
  

  Abdullah•340

  @ASH2001prince

  Posted almost 3 years ago

  Hello Wojtek Natkaniec congratulations for this project

  • I think you have a super responsive design here but I have something that I noticed like the main height if you remove the margin : 0 0; and add a padding : 20px the height of the page would be much better.
  • I`m not sure about this but I think working on the grid system here if easer I have done this project before and this is how I worked on the main container:

  .cards{
      display: grid;
      grid-template-columns: repeat(4, auto);
      justify-content: center;
      align-items: center;
  }
  

  Hope you find it helpful

  0
• 

  jom•130

  @Jmlsoteyza

  Submitted almost 3 years ago

  

  Order summary component

  1

  30
  

  Abdullah•340

  @ASH2001prince

  Posted almost 3 years ago

  Hello jom congratulations for your new project -I have some HTML tips for you:

  1. you can use the <section> or <article> elements to gather the elements like the <h1>,<p> and <img> together.
  2. instead of coping the <svg> you can import the image by using <img> element.
  3. <footer> usages for the last informations like names some links maybe and we can put the <address> attribute inside it as well.

  1

  Marked as helpful
• 

  Youssef Tadarti•90

  @YoussefTadarti

  Submitted almost 3 years ago

  

  Four Card Feature Section

  1

  30
  

  Abdullah•340

  @ASH2001prince

  Posted almost 3 years ago

  Hello @YoussefTadarti congrats for your first attempt I got some tips for you

  --HTML

  1. instead of adding the <div> as a container you should have put the <main> element.
  2. And the <div> card as a <article> and at the same time the <article> as a section,

  --CSS

  1. Its kinda tricky positioning the card by using the margin i think its better using this code:

  display: flex;
  justify-content: center;
  align-items: center;
  

  2. I did this project a couple days ago and this is how i designed the card => my main container:

  .cards{
      display: grid;
      grid-template-columns: repeat(4, auto);
      justify-content: center;
      align-items: center;
  }
  

  hope you find my feedback helpful.

  1

  Marked as helpful
• 

  VMahluza•10

  @VMahluza

  Submitted almost 3 years ago

  

  QR Code Challange

  2

  40
  

  Abdullah•340

  @ASH2001prince

  Posted almost 3 years ago

  Hello VMahluza welcome to the frontend mentor community _here are some tips for aligning the card.

  1. instead of using the <div> element everywhere you use the <main> as a container and <section> .
  2. for aligning the card you give the body selector these values:

  body{
          min-height: 100vh;
          display: flex;
          justify-content: center;
          flex-flow: nowrap column;
          align-items: center;
          align-content: center;
        }
  

  --keep it up

  1

  Marked as helpful
• 

  Siddhesh Parate•10

  @Siddhesh877

  Submitted almost 3 years ago

  

  Responsive QR code site using HTML and CSS only.

  1

  30
  

  Abdullah•340

  @ASH2001prince

  Posted almost 3 years ago

  Hello Siddhesh Parate congratulations for your first attempt here some tips that I can share it with you :

  1. In the html part you should use the <main> element as a container for all the elements instead of the <div>.
  2. if you want a responsive design you should`ve give a width and height values instead of auto.
  3. for a last thing aligning the body to the center would much easier, like so:

  display: flex;
  align-items: center;
  justify-content: center;
  

  hope you find it helpful

  0

  Marked as helpful
• 

  Abbas Roholamin•30

  @abbas-roholamin

  Submitted almost 3 years ago

  

  Design product preview card component using css flexbox and grid

  4

  60
  

  Abdullah•340

  @ASH2001prince

  Posted almost 3 years ago

  Hello Abbas Roholamin you did an amazing job with this project I did this project a couple days ago and I didnt use the grid in it so it didnt become responsive
  as it should be so I think that you did the best practice in here. -keep it up bro

  0

  Marked as helpful