Article preview, sass, react

Community feedback

• 

  P

  Zakhar Polikarpov•260

  @ZPolikarpovPosted 4 months ago

  Hello there. Well done on completing the challenge. There are a couple of things you can do to add more functionality and to use more of Reacts capabilities.

  FIrst of all instead of using straight up images on the "Share tooltip" wrap them in Links. This way you can navigate via the link by pressing on the image.

  While having a "show" property on your Sidepage can certainly be useful in more complex applications, you could shorten this by using conditional statements to render that part { isShareOpen && <Sidepage ... }. Since you've unpacked the Button image and used the SVG code directly: instead of assigning a class to change the button of the image you could instead change the fill property itself, which would change the color in a more apparent and quicker way: fill={color}

  Marked as helpful

  0