Menu

#79 reduce prompting for going to Europe

closed-accepted
nobody
None
5
2011-06-14
2011-06-04
No

When you use "Go To" to direct your ship to a destination that is a considerable distance off, there is a good chance the ship will cross the high seas (for the most direct route). This is particularly likely if you're sending your ship between the poles, for example.

When your ship crosses into the high seas, you are asked whether you would like to go to Europe. However, when you have already set your Go To route to somewhere in the new world, that question doesn't make sense.

The attached patch adds a check, making sure the Go To route (if any) points to the high seas before prompting. The patch is against trunk.

Of course class Tile could potentially use a member isType(enum {Sea, HighSea, Land, ...}) or something like it, but that is for the maintainer to decide.

Discussion

  • Bastiaan Jacques

    fix

     
  • Mike Pope

    Mike Pope - 2011-06-04

    Good idea. I will commit this soon.

     
  • Mike Pope

    Mike Pope - 2011-06-14
    • status: open --> closed-accepted
     
  • Mike Pope

    Mike Pope - 2011-06-14

    A variant of this patch was committed as svn.8516.

     

Log in to post a comment.

MongoDB Logo MongoDB