Skip to content
OverviewSolutions
18
Comments
6

Krutagna Patel

@krutagna10India390 points

Hello I am Krutagna Patel I am currently focusing on upgrading skills.

I’m currently learning...

Javascript and SCSS

Latest solutions

View more solutions

Latest comments

  • Krutagna Patel•390
    @krutagna10

    You can use picture tag instead of using two images inside a div.

  • Krutagna Patel•390
    @krutagna10
    1. Increase the font size of the page, its really small on mobile screens.
    2. You can add feature to remove the notification when the user clicks on single message.
    3. Name variables in lower case, with words separated by hyphens Example : use grayish-blue instead of Grayishblue
    4. You have made spelling mistake while setting the box-sizing on all elements
    5. Add some comments in the code so that it will be easy for the people who are reading your code.
    Marked as helpful
  • Krutagna Patel•390
    @krutagna10

    Hello @smayrant I wanted to know how did you make designo favicon.

  • Krutagna Patel•390
    @krutagna10

    @vanzasetia I have one question, if you are using scss then why are you not nesting the selectors ?

  • Krutagna Patel•390
    @krutagna10

    1. You have to make the display of those images from inline to block to remove those gaps.

    2. You can always add this code at the starting of your css file to make images easier to work with.

     img {
  display: block;
  max-width: 100%;
}
    Marked as helpful
  • Krutagna Patel•390
    @krutagna10

    👾Hello @Cydr4l, Congratulations on completing this challenge!

    I think you can add a paragraph saying "Kindly select a rating" when the user does not select any rating.