Looks good to me, assuming the unlabeled resistors are within a reasonable value. Not sure about the one on drain. You shouldn't need those.
One tip though: The use of the PIC isn't all that great. What I mean is, PORTB looks much better for input with weak pull-up and int-on-change. Many people put resistors on switches, but you can save space/wire by using WPU. Just replace the 5V with ground, and in code enable WPU on PORTB and reverse the branch logic.