Interactive Rating Component

Community feedback

• 

  Chukwuka Eze•310

  @ezechukaPosted over 2 years ago

  I checked your CSS code and it's very neat, I like it. If you want to write less and easier CSS I suggest learning/using SASS (that is if you don't know about it before), it's a css preprocessor with cool features like nesting, functions, inheritance.

  For example: this code snippet from your style.css:

  .container-thanks h2 {
      padding-top: 1rem;
    }
  
    .container-thanks p {
      margin-bottom: 1rem;
      text-align: center;
    }
  

  can be written or nested as:

  $top-padding: 1rem;
  .container-thanks {
        h2 {
            padding-top: $top-padding;
       }
       p {
          margin-bottom: $top-padding;
          text-align: center;
       }
  }
  

  Marked as helpful

  1

  ---

  

  Skippy•110

  @skippysworldPosted over 2 years ago

  @ezechuka Thanks! Appreciate your point pf view!

  Yea I've heard about less sass before, but I aim for what I think is right approach - to learn basics well before pre-processors or frameworks. I'm afraid it would confuse me more in the long term.

  I dont necessarily think it's too much CSS rather than I could use more of common classes. To describe better - I have feeling I duplicated code that might have been implemented and reused in better manner :)

  1