Does component name give a clear idea on the functionality it provides?
Does the file use components efficiently, is there comments that can help someone understand what's being rendered.
Do the classnames being used for the elements make sense? Is there ambiguous classNames such as "text1". A better className could be "intro-paragraph" for example.
Is the file using the minimal amount of css classNames possible? Or can some of the classNames be merged together for better readabililty?
Are all the imports being used?
We want the code to be clean and readable enough for the client to be able to make updates on her own.
Button should be placed under the left paragraph in the support page. Might have to set up a backend express server to help with using the making requests for the API. If you need any details/credentials from the client please let me know.
Things to note:
--> use "===" instead of "=="
--> also if we're doing 'class ___ extends React.Component { .... ' I believe we don't have to import {Component} from 'react'. We only additionally import {Component} if we're doing 'class ___ extends Component' I think.
--> many unused variables.