That looks good to me. You have a good ratio of track to scenery and structures. In my opinion, many people fill the layout with track and don't leave enough room for a town, industries or scenery. You might consider making the town a little larger and reducing the area devoted to pine trees. Also, add another switch and make the (front) siding a passing siding. That would give you the option of having two trains on the layout and running one at a time.
MELGAR