It's just bad practice, a bad habit to get in to.
It's not all that important with this design, though it could make soldering harder, and [if the PCB manufacturer was less than perfect] cause failures or reliability problems, as all topside component connections are reliant on the plating being good and solder fully flowing.
And with more complex, higher frequency or small signal boards, one of the purposes of a ground plane is to provide screening between the components and other signal tracks that pass underneath them.
Having the signal traces on the top side totally defeats that.
Surface mount and multilayer or stripline / dead bug etc. boards are obviously different, but for conventional components I have never seen a single commercial board with the tracks swapped. Some at one time occasionally had an image of the underside printed on the top to show connectivity, but the copper was still on the underside.