Skip to content
OverviewSolutions
23
Comments
23
Mohtashim Ali
@MohtashimAli85

All comments

  • Mohtashim Ali•510
    @MohtashimAli85Posted about 2 years ago
    This comment was deleted over 1 year ago
    0
  • Mohtashim Ali•510
    @MohtashimAli85Posted about 2 years ago
    This comment was deleted over 1 year ago
    1
  • Mohtashim Ali•510
    @MohtashimAli85Posted over 2 years ago

    loved it !!

    1
  • Mohtashim Ali•510
    @MohtashimAli85Posted over 2 years ago

    Kindly resolve the accessibility issues. Other than that Its great and just fix the bg colors to match the design.

    0
  • Mohtashim Ali•510
    @MohtashimAli85Posted over 2 years ago

    I reviewed your code and website both are good. but the background images position you need to play with values to set them ig. Over all you doing great. :)

    Marked as helpful
    0
  • Mohtashim Ali•510
    @MohtashimAli85Posted over 2 years ago

    The things I noticed

    • To make body take full height give body and html tag height 100% html,body{ height :100%}
    • Learn grid you can easily center a div with it like this
    main{display:grid;
place-items:center;
}
    • And its static the daily, weekly buttons don't work. Also share the repo to review your coding. Happy coding :)
    0
  • Mohtashim Ali•510
    @MohtashimAli85Posted over 2 years ago

    Frontend Looks great! But the code I think is messy. It can be a lot of simpler. I think you did it for learning but I would like you to make it as simple as you can! happy coding :)

    Marked as helpful
    0
  • Mohtashim Ali•510
    @MohtashimAli85Posted over 2 years ago

    Looks Great. I think it's time for you to start higher level projects.

    Marked as helpful
    1
  • bunee•2,020
    @buneeIsSloSubmitted over 2 years ago

    Hi! It's been a good minute since I posted a solution on here. I have been teaching myself ReactJS and wanted to create a small project so, I chose to solve this newbie challenge for some practice.

    However, I was unable to complete this challenge on my own. Thanks to this solution I realized where I was going wrong and managed to create my solution.

    I have no questions for this one, but I'm open to any tips and feedback you have for me :)

    Click here to view the live site

    Click here to view the code

    ⭐ Interactive Rating Component | React | SCSS | Webpack

    3
    Mohtashim Ali•510
    @MohtashimAli85Posted over 2 years ago

    btw can you give tips regarding seamless animations and anything else :3

    0
  • bunee•2,020
    @buneeIsSloSubmitted over 2 years ago

    Hi! It's been a good minute since I posted a solution on here. I have been teaching myself ReactJS and wanted to create a small project so, I chose to solve this newbie challenge for some practice.

    However, I was unable to complete this challenge on my own. Thanks to this solution I realized where I was going wrong and managed to create my solution.

    I have no questions for this one, but I'm open to any tips and feedback you have for me :)

    Click here to view the live site

    Click here to view the code

    ⭐ Interactive Rating Component | React | SCSS | Webpack

    3
    Mohtashim Ali•510
    @MohtashimAli85Posted over 2 years ago

    Loved the animation part <3 Also code is clean !

    1
  • Mohtashim Ali•510
    @MohtashimAli85Posted over 2 years ago

    Neat ! Other than that I would recommend you to make components which you did, but make more. Try to breakdown the website layout into proper components. It will help you a lot. For example Navbar component HeroSection component Banner component also instead of footersection name i would recommend just footer Though In the end we just see the output. But following these will definitively ease you. These are just my opinions. Enjoy coding

    Marked as helpful
    1
  • Mohtashim Ali•510
    @MohtashimAli85Posted over 2 years ago

    learn about transitions Say we have a anchor tag whose color changes to red on hover for this /CSS Transition Syntax/

    selector {
 transition: property duration timing-function delay|initial|inherit; 
}
a{
transition: color 0.2s ease-in;
}
a:hover{
color:red;
}
div:hover {
  background-color: #000;
  transition: background-color 400ms;
}

/* Or control over animation with transition timing */
div:hover {
  background-color: #000;
  transition: background-color 400ms ease-in-out;
}
    1