The landmass looks quite good, I do see some odd river stuff going on, mainly 2 instances of a "river island" that is a river that splits then joins again.. which is basically nonexsistant in the natural world except for river deltas, which are a special concept... they could easily be changed into lakes and make complete sense... there is also one river on the south coast that branches out to sea (starting with one river, and splitting to the sea) which is kinda the opposite of what rivers do, they're like trees, growing from the ocean inland.. so they start out with one large "trunk" on the coastline, the main river, then split off into "branches" as they go inland... obviously if these things are imperative to the story, then you can just make em so... but if you looking for a bit more of a geographically sound world, then they should probably be changed. Also, lakes could be a good addition... need not be of huge size.. but lakes are pretty common, so in a continent there should be any number of them.

Something to remember with cities is that they really need a purpose as to why they are at a location, a city makes a lot more sense when you can justify it's exsistance... rivers are probably the most common way to locate a city since it is so vital to life... but other such things are resources (lumber, farming, ore, etc), defenseive locations, strategic locations (an outfitter city at the edge of a mt pass)... There's nothing in your map that looks off, but in the long run, it's surprizing how much planning in the beginning can help out in the end... for instance, city names can come about from the use they were founded.. Bournemouth, an city on the south coast of england literally means river's mouth... A river fjord, port, pass are all good starting points for city names... say them in "native tongue" and you can get more colourful names in themselves... but knowing a cities imports/exports can help out quite a bit too.. if i city needs lumber where does it come from? the closest city that exports lumber... so thats a good indications of decent trade relations between those cities... fair-to-large sized caravans will be travelling that route, therefor a decently upkept road would be in their interest... cities who must go through a mt pass to do trade are going to need to be fairly self reliant, and only import luxury goods or other things that aren't life or death...

World building is in many ways a huge project... for many of us it's a labour of love... and it can give you some fresh insights on history and other things you've never thought about...

You can go as detailed or as generic as you want with a world, but as a general rule the more detail you put into the world, the more it can contribue to your story... world histories, factions, riches and locals are all great initiators for a players adventure, and the more work you put in before hand becomes less work and improv you need to do just before or even during a gaming session.