My comments on your "schematic", some of my comments duplicate some of those which have been made already.
The standard of drawing is awfull with wires criss-crossing here and there for no good purpose, makes it very hard to read.
There is no power to the PIC power pins.
Why is the power shown connected to MCLR.
D9 serves no purpose, should not be connected to the crystal, and would probabbly stop the oscillator anyway.
There are no load capacitors for the crystal, the oscillator will not work correctly without them.
Why are RA0 and RA1 connected to 0v? If RA0 or RA1 are configured as outputs, you will short out the supply and probably damage the PIC. Or are these supposed to be inputs, it is not clear the way they are intertwined with the LEDs.
There are no current limiting resistors on the LEDs.
JimB