I’m hoping to review the interns’ code before falling asleep, but it might not happen. Looking through things briefly this afternoon, there are three things I noticed.
First, there’s a tendency to look to loops instead of higher-order functions. This is reasonable since the APIs are still unfamiliar and loops are the known tool.
Second, HTML elements are sometimes chosen for their visual appearance and not because of their appropriateness.
Third, labels are often picked because they describe an implementation detail in the current context, not because they make the most sense for the content they hold or describe.