#99 Single right click pathfinding

closed-accepted
None
5
2012-10-30
2012-09-27
No

From Ovo on the forum http://exult.sourceforge.net/forum/read.php?f=1&i=412249&t=412249

This makes pathfinding work on single right click instead of double click.

Might be worth adding as an option.

Discussion

  • Dominik Reichardt

    assigning this to Malignant. Can you check whether the bool right_dragging is really needed? For *normal* operation it seems to work fine without it. But as you are some kind of expert in the 'charmed' area you might know better.
    I could also need your help in making this an option, so that you can chose "Right-click to pathfind" with the options "one, double, disabled" (or similar). You are way better at those things than me.
    Don't feel pressured, I'm just assigning this to you to get your attention :)

     
  • Dominik Reichardt

    • assigned_to: nobody --> malignantmanor
     
  • Dominik Reichardt

    I've made some progress in doing it myself. See the attached second diff.

    Problems:
    - when single click is selected a doubleclick pathfinds as well
    - don't know how to make exult remove the now invalid "config/gameplay/allow_double_right_move" setting. I think we can do this, I just don't know how

     
  • Dominik Reichardt

    to remove the old cfg something like this? But where?

    if (config->key_exists("config/gameplay/allow_double_right_move")) {
    config->value("config/gameplay/allow_double_right_move", doubleright_move?"yes":"no", false);
    config->remove("config/gameplay/allow_double_right_move",false);
    config->set("config/gameplay/allow_right_pathfind", right_pathfind-1, false);
    }

     
  • Dominik Reichardt

    • assigned_to: malignantmanor --> nobody
     
  • Dominik Reichardt

    new patch, single click will not also allow double click now.

    I'm thinking about leaving the old setting orphaned...

     
  • Dominik Reichardt

    newest patch adds the transformation of the old cfg entry and all the help from wjp.

     
  • Dominik Reichardt

    • assigned_to: nobody --> dominus
    • status: open --> closed-accepted
     

Log in to post a comment.