feature: share social #220
Conversation
|
Já tem um PR com compartilhamento: #130 |
O PR que você citou tem alguns problemas de layout que não foram refinados. Esse aqui parece mais completo. |
| const shareButtons = [ | ||
| { Component: WhatsappShareButton, Icon: WhatsappIcon }, | ||
| { Component: EmailShareButton, Icon: EmailIcon }, | ||
| { Component: TwitterShareButton, Icon: TwitterIcon }, | ||
| { Component: TelegramShareButton, Icon: TelegramIcon }, | ||
| { Component: InstapaperShareButton, Icon: InstapaperIcon }, | ||
| { Component: LinkedinShareButton, Icon: LinkedinIcon }, | ||
| ]; |
There was a problem hiding this comment.
É uma boa prática mover variáveis constantes para fora do componente. Do jeito que está, a variável shareButtons é criada a cada instância do componente sendo que poderia ter sido criada apenas uma vez e reutilizada em todo lugar que esse componente é usado.
Nesse caso não há problema de performance, mas em outros pode afetar.
A ideia dos abrigos é compartilhar com pessoas próximas e uma forma fácil de fazê-lo é através da feature de share.