React App using Styled Components. Flexbox for responsive.

Community feedback

• 

  Sean Red•635

  @seanred360Posted about 2 years ago

  What do you specifically mean by " add active states" which element needs it and what does it look like? I can help you if I understand it better.

  ignore this

  This looks like a short commentThis looks like a short commentThis looks like a short commentThis looks like a short commentThis looks like a short commentThis looks like a short commentThis looks like a short commentThis looks like a short commentThis looks like a short commentThis looks like a short commentThis looks like a short commentThis looks like a short comment

  0

  ---

  

  Moises Silva•90

  @knitaxdPosted about 2 years ago

  @seanred360 Thanks for your answer!!! In the features section, I need to put an active state (border-bottom: red)when you click one of the tabs. I did it adding a class but idk if this is ok with styled components

  0

  

  Sean Red•635

  @seanred360Posted about 2 years ago

  @knitaxd There is no rule that says you cannot do that. Styled components is just a tool for writing CSS faster. You just need to do this.

  1. Does it work?
  2. Can other devs understand your code without having to search in many different places? For example did you write inline styles, use a css file, and also use styled components at the same time for no good reason? Keep things organized.
  3. Did you use meaningful names and eliminate mental mapping?

  Marked as helpful

  0