antonio-ramadas / antonio-ramadas/palbum
Refactor App.js
Open
frontend
good first issue
- Dominant language
- JavaScript
- Stars
- 9
- Forks
- 0
- PR merge metrics
- No merged PRs in 30d
Description
[App.js](https://github.com/antonio-ramadas/palbum/blob/master/src/App.js) is doing more than it should. It populates the page (search bar and table).
However, whenever there is an update to the list (e.g., a new album is added) the whole component is rendered. As you can guess, there is no need for it.
Also, with a refactor, the logic would no longer be on a single file and its size will most likely be reduced.
I suggest having one component for the search bar, another for the table and App.js simply calls them on the `render`method.
Contributor guide
No contributing guide indexed for this repository
Assessment
This issue has not been assessed yet.