What GRJ said. For a first try it's quite good!
If I understand it, your board design program does not have 3-D rendering (?) so it can be hard to visualize how the parts interact without actually assembling a unit. Back in the day, when designing a board using through-hole parts what I found useful was to place the parts on a 0.1" grid perforated project board. Of course this kind of measure-twice, fabricate boards-once practice has been turned on its head with what has become zero fixed $ penalty to fabricate revisions.
Photo shows a couple other ideas. Those screw-terminal connectors come in 0.1" pitch. I see you're using 0.2" pitch which obviously take up a good chunk of real-estate. I realize you already have the connectors but maybe next time if size matters. Also, for space savings nothing wrong with standing up the resistor like black diode in above photo with pads spaced 0.1".
Separately, on your latest revision when you install the component you cover the silkscreen. Your software should allow you to drag the white silkscreen reference designators (R1, RV1, J1, etc.) or labels/symbols (AC IN, +, etc.) so you can see them after the parts are mounted. Note on GRJ's board how lettering is outside the footprint of the parts. And for your RV1, it's usually customary to have a square-pad for pin 1; or for symmetrical parts like this (can be installed either way) at least some silkscreen indictor showing notch, tab or in this case maybe a small circle showing which end has the adjustment screw.