All the above.
Schematic:
If something goes down to ground then it should show it goes "down" to ground. Up to supply etc.
Try to get your labels (component name, value etc.) all reading left to right so the reader does not have to look sideways etc.
Also get them the same size, CR1+CR2 are huge compared to others.
Rotate J1 90 to the right.
Also rotate VR1 so the labels are readable left-right, in fact the whole symbol would be better if the IN was on the left edge, the OUT on the right edge and GND on the bottom edge.
And connect the +12vs so that J1 feeds into VR1, there is no point having the 12v, add a text label if you want to identify the voltage.
MOS1 is a different scale to the other symbols, consider making it smaller.
The LED's, if you have a line through them you can rotate them 180 and not realize it, diodes are best with no lines through them.
R9 has 22 inside the symbol.
But heck, good try - its still readable.
The PCB:
C3 should be right next to the input pin of the regulator with thicker tracks.
C4 by the output pin (I cannot even find C4???)
You have no protection on your input, no reverse polarity diode nor fusing?
What is "power" ?
C5 & C6 +pins connect to R3 and SS2 Mode pins (+5v). On your schematic this is not the same net for IC1 pin 6 to C6 (not shown as electrolytic) and R8? Thier placement is nowhere near where they should be (wherever that is - but its not where they are now).
Either your labels are not next to the components they belong to, or you have some serious misplacement issues, take CR1 for instance, its next to P1? (is that JP1) yet the schematic shows it as connected to JP2.
I'm going to stop here as it clearly does not match the schematic for me - you need to ensure that it does.
What software are you using?
Either way, welcome into the hobby, please dont take this as bad criticism but helpful criticism (is that actually a thing?).
Enjoy the hobby, learn more, do more, enjoy it more. (Beer helps).