Social Links Profile

Community feedback

• 

  P

  Nazarii Sabadash•50

  @sabbadashPosted about 2 months ago

  Hi! I have got a few ideas about your soulution.

  • Use semantic HTML by replacing the div containers with more meaningful elements like main and article

  • Consider using a tags styled as buttons instead of button elements since these are links

  • Instead of using IDs (#a, #b, #c, etc.) for buttons, consider using a single class and margin-top. This reduces code repetition

  • The box-shadow on the container could be softer for better visual appeal

  • Consider using CSS custom properties (variables) for colors to maintain consistency

  • The layout could benefit from CSS Grid or Flexbox for the buttons section

  You're doing great! Keep going ;)

  Marked as helpful

  1

  ---

  

  RoHaN•150

  @Rohan-DahalPosted about 2 months ago

  @sabbadash Thanks for your feedback.

  0