Quick code review.
Stick to one type of class names (
recentLink). I use
my-cool-class. And btw, writingeverythinginoneword should be avoided (
And I think the font you’re trying to use is called Verdana
If you make both requests when app loads and save both to state you can switch instantly between both lists and no further network requests required (you could add functionality to periodically fetch in the background, say, every hour). Also FCC has limit on their API so this is the best approach here.
Accessing elements with
document.getElementBy... is not a very “react way”.
I made some refactorings in your code (I’m not saying it’s the best way to do it, just one of the ways):
Also there is ‘Tidy JS’ option in CodePen to help format your code.