<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On 11 July 2017 at 07:09, James Paige <span dir="ltr"><<a href="mailto:Bob@hamsterrepublic.com" target="_blank">Bob@hamsterrepublic.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="HOEnZb"><div class="h5"><br><br>On Tuesday, July 11, 2017, Ralph Versteegen <<a href="mailto:teeemcee@gmail.com" target="_blank">teeemcee@gmail.com</a>> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On 11 July 2017 at 03:45, James Paige <span dir="ltr"><<a>Bob@hamsterrepublic.com</a>></span> wrote:<br><blockquote style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex" class="gmail_quote"><div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote"><span>On Mon, Jul 10, 2017 at 8:35 AM, Ralph Versteegen <span dir="ltr"><<a>teeemcee@gmail.com</a>></span> wrote:<br><blockquote style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex" class="gmail_quote"><div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote"><span>On 8 July 2017 at 11:13,  <span dir="ltr"><<a>subversion@hamsterrepublic.co<wbr>m</a>></span> wrote:<br><blockquote style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex" class="gmail_quote">james<br>
2017-07-07 16:13:27 -0700 (Fri, 07 Jul 2017)<br>
78<br>
When pathing to an NPC, the destination indicator moves smoothly with the NPC<br>
---<br>
U   wip/game.bas<br></blockquote><div><br></div></span><div>I see that here you offset the indicator by the footoffset, but only when pathing to an NPC.<br></div><div>The footoffset also ought to be used when pathing to a tile, and also to determine which tile you clicked on. If you set the footoffset to something really large you'll see that not including it isn't right.<br><br></div></div></div></div></blockquote><div><br></div></span><div>That is a good point. I'll fix both those things<br></div><span><div> </div><blockquote style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex" class="gmail_quote"><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><div></div><div>Also, maybe a shadow-shaped ellipse at the bottom of the tile would be another good alternative?<br></div></div></div></div></blockquote><div><br></div></span><div>My plan is to make the indicator a user-definable slice collection. The ellipse will just be a default.<br></div><span><div> </div><blockquote style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex" class="gmail_quote"><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><div><br></div><div>Semi-relatedly, I've just realised something rather amusing: if you enable mouse 
controls, the player can cheat any maze by simply clicking on the exit, 
and the engine will path there! Also, it was amusing to discover that if there's a gap anywhere in your wallmap and you click out-of-bounds, the engine will happily path you around the map and through the wallmap bug. I think we should probably put a fairly small (and per-map customisable?) limit on the max hero path length to avoid both problems. (For example then you can set it low on maze maps and high if testing your own game for wallmap bugs)<br></div></div><br></div></div></blockquote><div><br></div></span><div>That is a good idea. I'll start with adding a global max hero path length. We can see about per-map overrides later.<br><br></div><div>And of course, it will be easily accessible from the F8->Mouse Options debug screen for testing.<br></div></div></div></div></blockquote><div><br></div><div>That reminds me of the other thing I meant to mention: I noticed that the mouse debug menu modifies the original general.reld.<br><br></div><div>This only makes a difference if you're either using Test Game, or running a .rpgdir. We really need to stop playing .rpgdirs inplace and go back to copying them to playing.tmp (except on Android... and OSX if its in a .app?) because I'm sick of having to revert vikings.rpgdir!<br><br></div><div>If you're running under Custom then changes in the menu only last until Custom overwrites it when it next modifies it.</div><div>It's not obvious how to make it better behaved under Custom but I guess it could be possible to merge in changes from custom to subtrees that haven't been modified, by using CompareNodes to see what's changed... sounds really low priority.<br></div></div><br></div></div></blockquote><div><br></div></div></div><div>Maybe this just means we need to wrap write_general_reld() in #IFNDEF IS_GAME</div><div>?<br></div></blockquote><div><br></div><div>After looking into it, that will fix Game modifying general.reld when you use the mouse debug menu while playing. (It'll still write it when upgrading a .rpgdir, of course). It also fixes the lump permanently being modified if you Test Game and then quit Custom without modifying any general data (it doesn't appear to always save general.reld when quitting, unlike gen(). Which might be a source of bugs. It'll still mean the mouse options get overwritten by Custom all the time.<br></div></div></div></div>