Few things I noticed.
1: your move routes have a few repeating moves, such as the "turn right, wait 3 frames" which happens twice in the first. Same thing for the "turn left, wait 3 frames" on the second page.
2: The second page, the move routes aren't set for "Wait for Completion" which means the event will go through the list without waiting for the moves to complete. This means that the player will move down immediately, and Turn A Off will start immediately as well.
The second thing is important because it messes up your move routes and changes the page before it finishes.
The first page also sets the curtain to "Through" but the second page is not set to "Through" by default (via the box on the lower left). It also changes the graphic to an open curtain, but the second page's default image is closed. This means that if you leave the map and return, it will be on the second page, the curtain will be closed and it will look as if it's closed when it should actually be open.
Here's the steps to fix this:
Page 1: leave as is.
Page 2: Set to "Through", change graphic to the open curtain.
Set both move routes to "Wait for Completion"
Delete the extra moves in the first move route, also delete both "Through" commands.
Create a new move route after the player moves down to change the curtain to "Through Off".
If you follow this exactly, then the player will open the curtain and move forward. But when they close it, no matter where they are, they will move down twice. It would be better if you delete both Player move routes so that all it does is open and close the curtain.
Edit: your method right now is more appropriate than Skot's. Skot, you can actually loop it by using only one switch, unless you're using both switches in different ways for other events.