takenote icon indicating copy to clipboard operation
takenote copied to clipboard

Refactor, unnecessary ActionButton render

Open Quintis1212 opened this issue 2 years ago • 1 comments

Hello everyone ) I made a little refactor for AppSidebar component . I pass props from TakeNoteApp to AppSidebar directly , so AppSidebar does not call useSelector and recive props from parent. Also I discovered that ActionButton re-renders after every click on menu or on text changing in note. I decided to prevent unnecessary re-renders of ActionButton with PureComponent and children props. Please feel free to comment on how we can make it better

Quintis1212 avatar Aug 29 '22 13:08 Quintis1212

@code128 @dagda1 @yogan @xboston

Quintis1212 avatar Aug 29 '22 14:08 Quintis1212