Personally oppinon here so take it as that only.
1) the mountain range are too straight make it more "random" a bit wide in some places and narrow in others. Remember mountains are made either by the gods (fantasy) or by tectonic plates pushing the ground upward. If the latter is the case then imagine how the tectonic plate have moved and model the mountains after that.

2) Rivers flow toward the sea - Cool enough - however they should raise from the higher grounds. Specially the one running thru penbridge (sp?) should come from some sort of platau. This elevation could be made by adding "cliffs" to the waterfront.

3) water front - pretty straight forward however add a few lesser islands or rock formation to make it more realistic

4) City location - Assuming this is a feudalism type of society (as most fantasy) the settlement generally were located near water for easy travel, water mills and fishing. These were normally major settlement making faming community somewhat lesser - you could reflect that with different sizes on your towns. Also I understand this is for a story but who would build a fortress on the "wrong side" of the mountains - Fortress are build for protection (and pride) so why did the land owner build it where they did. Consider that when placing the fortress.

5) Overall colour scheme - You are using standard CC colour scheme which in itself is fine - but try to add some variation to the river + roads and maybe some more barren/rocky type of colour near the glacier as the plains wouldnt be the same all over - also consider woodlands somewhere... maybe pine in the north?

Anyway good fundament to build on and lots of space for adventuring thru your tales of the forgotten ages.