Topic: Glitches - Mouse.Mode issue and Unwanted Character Animations

[KodiakBehr]

Hey, I've uncovered two bugs that I cannot reproduce at will which makes it extremely difficult to ascertain what is going wrong.

1.  Sometimes, without any obvious explanation, the character will play a loop unrelated to his walk-cycle when walking.  Any guesses on what can cause that?
2.  The inventory system doesn't deviate too much from the default Sierra system.  Despite the fact that the code says:

Code: Adventure Game Studio

1. function show_inventory_window ()
2. {
3.   gInventory.Visible = true;
4.   // switch to the Use cursor (to select items with)
5.   mouse.Mode = eModeInteract;
6.   // But, override the appearance to look like the arrow
7.   mouse.UseModeGraphic(eModePointer);
8. }

Occasionally (but inconsistently) the inventory will appear but the cursor will still be the walk cursor.  Any ideas on what could cause that?

Thanks.

KB

[Khris]

Are you using any templates or modules?

[KodiakBehr]

Default game template, minus the keyboard movement, and a custom module I don't fully understand that generates text as an overlay...though I don't see how it would impact either bug.

The first bug I mentioned appears to play the last character animation loop, regardless of what it did, in place of walking forward.

[Khris]

So the character's NormalView contains additional loops unrelated to walking?
I'd use a different view for that.
Also make sure you've turned off Diagonal Loops in the character's properties.

As for the second issue, something in the code must cause that. There's not much we can do without looking at at least all the repeatedly_execute functions of your game.

[KodiakBehr]

Yup, I didn't realize that was a bad practice.  Though for what it's worth, diagonal loops was left on, and turning it off appears to have resolved the issue.  Many thanks for that.

Regarding the second issue, in the global script, we have this repeatedly executing --

Code: Adventure Game Studio

1.  
2.   int old_mode;
3.   bool was_over_inventory_button;
4.   bool was_over_cellphone_button;
5.   bool was_over_menu_button;
6.  
7. function repeatedly_execute_always() {
8.  
9.  
10. GUIControl*gc = GUIControl.GetAtScreenXY(mouse.x, mouse.y);
11.   if (gc == btnIconInv && !was_over_inventory_button) { // mouse just went over the button
12.     was_over_inventory_button = true;
13.     old_mode = mouse.Mode;
14.     mouse.Mode = eModePointer;
15.   }
16.  
17.  
18.   if (gc != btnIconInv && was_over_inventory_button) { // mouse just went away from button
19.     was_over_inventory_button = false;
20.     if (mouse.Mode != eModeUseinv) mouse.Mode = old_mode;
21.   }
22.  
23.   if (gc == btnCellphone && !was_over_cellphone_button) { // mouse just went over the button
24.     was_over_cellphone_button = true;
25.     old_mode = mouse.Mode;
26.     mouse.Mode = eModePointer;
27.   }
28.  
29.  
30.   if (gc != btnCellphone && was_over_cellphone_button) { // mouse just went away from button
31.     was_over_cellphone_button = false;
32.     if (mouse.Mode != eModeUseinv) mouse.Mode = old_mode;
33.   }
34.  
35.   if (gc == btnIconMenu && !was_over_menu_button) { // mouse just went over the button
36.     was_over_menu_button = true;
37.     old_mode = mouse.Mode;
38.     mouse.Mode = eModePointer;
39.   }
40.  
41.  
42.   if (gc != btnIconMenu && was_over_menu_button) { // mouse just went away from button
43.     was_over_menu_button = false;
44.     if (mouse.Mode != eModeUseinv) mouse.Mode = old_mode;
45.   }
46.  
47.  
48. if (gCellphone.Visible==true){mouse.UseModeGraphic(eModePointer);
49. }
50. }
51.  

And in the module script, we have this repeatedly executing --

Code: Adventure Game Studio

1. function repeatedly_execute_always()
2. {
3.     isNew = true;
4.     if (oLabel != null && oLabel.Valid) {
5.       oLabel.Remove();
6.       isNew = false;
7.      
8.     }
9.    
10.     if (isNew) {
11.       moveTimer = 0.0;
12.     }
13.     String s = Game.GetLocationName(mouse.x,  mouse.y);
14.     if (s == null || s == "") return;
15.    
16.     int width = _clamp(GetTextWidth(s, Font), 0, MaxWidth);
17.     int height = GetTextHeight(s, Font, width + 2);
18.    
19.     float smooth = _smoothstep(0.0, 1.0, moveTimer);
20.    
21.     int drawY = FloatToInt(_lerp(IntToFloat(height), 0.0, smooth));
22.      //int drawY = FloatToInt(Maths.Sqrt((1.0 - moveTimer) * 100.0), eRoundNearest);
23.     moveTimer = saturate(moveTimer + 0.1);
24.    
25.      
26.     if (!SlideIn) drawY = 0;
27.    
28.     sprite = DynamicSprite.Create(width, height, true);
29.     DrawingSurface *surf = sprite.GetDrawingSurface();
30.     surf.DrawingColor = Color;
31.     surf.DrawStringWrapped(0, drawY, width + 1, Font, eAlignLeft, s);
32.     surf.Release();
33.    
34.     float targetx = IntToFloat(_clamp(mouse.x - width / 2, 0, System.ViewportWidth - width));
35.     float targety = IntToFloat(_clamp(mouse.y - height, 0, System.ViewportWidth - height));
36.    
37.     float x, y;
38.    
39.    
40.     if (isNew || !SpringOn){
41.       x = targetx;
42.       y = targety;
43.  
44.     }
45.     else
46.     {
47.  
48.  
49.         x = lastMX - (lastMX - targetx) / 4.0;
50.         y = lastMY - (lastMY - targety) / 4.0;
51.     }
52.      
53.       lastMX = x;
54.       lastMY = y;
55.    
56.     //Display("Showing %s at: %d %d", s,  x,  y);
57.    
58.       if (!gInventory.Visible && IsInterfaceEnabled()) {
59.         oLabel = Overlay.CreateGraphical(FloatToInt(x, eRoundNearest), FloatToInt(y, eRoundNearest) + Offset, sprite.Graphic,  true);
60.       }
61. }
62.  
63.  

Grateful for any thoughts you might have.  I appreciate that showing labels as overlays is seen as being a bit dodgy.

KB

[Khris]

Well, in theory it's fine to put additional animation loops in the NormalView, but I'd skip loops 4-7 in that case. Unless you're positive that you won't add diagonal walk cycles at some point in the future.

Right, I remember the thread about changing the cursor over the buttons.
I guess it's pretty safe to say that the second issue is caused by that.
What's in btnIconInv_OnClick?

[KodiakBehr]

Code: Adventure Game Studio

1. function btnIconInv_Click(GUIControl *control, MouseButton button) {
2.  
3. btnCellphone.Visible=false;
4. if (player.NormalView==1){player.Animate(9, 0, eOnce, eBlock, eForwards);} // if our protagonist is normal sized, show him pulling out his tablet before loading the inventory
5. else if (player.NormalView==6){player.Animate(10, 2, eOnce, eBlock, eForwards);}  //if our protagonist is tiny in the city view, do the same
6.  
7.   show_inventory_window();
8. }
9.  

Which leads us to...

Code: Adventure Game Studio

1.  
2. function show_inventory_window ()
3. {
4.   btnIconMenu.Visible=false;
5.   btnIconInv.Visible=false;
6.   gInventory.Visible = true;
7.   // switch to the Use cursor (to select items with)
8.   mouse.Mode = eModeInteract;
9.   // But, override the appearance to look like the arrow
10.   mouse.UseModeGraphic(eModePointer);
11. }
12.  
13.  

And then it's turtles all the way down.  I can't see any scenario where upon loading the game inventory the cursor is anything BUT an interact cursor disguised as a pointer, yet occasionally it shows up as my WalkTo icon for reasons beyond my understanding and beyond my ability to replicate.

If there's nothing obvious in any of this, and you're still interested/willing to look at it, I'm happy to share the entire source code with you.  Maybe I'd learn a thing or two on what I could be doing better.

[Khris]

The explanation is actually pretty simple:

The way the rep_exe code works is:
-store the current mouse mode when player hovers over btnIconInv, switch to pointer
-upon leaving the button, restore previous mode

Any time you want to open the inventory:
-mouse is moved over button, current mouse.Mode is stored in old_mode, mouse.Mode is changed to Pointer
-show_inventory changes mode to eModeInteract but switches graphic to Pointer, i.e. user doesn't notice the change
-user starts operating the inventory GUI, the mouse leaves btnIconInv in order to do so, and old_mode is restored

Now, if old mode was eModeInteract, nothing changes. But if old_mode was eModeWalkto, that's restored, per the instructions in rep_exe.

The solution is to disable the rep_ex instruction. Make sure that show_inventory is below the rep_exe and its variables, then add this line to it:

Code: Adventure Game Studio

1.   was_over_inventory_button = false;

[KodiakBehr]

That did the trick.  I cannot express my gratitude for picking up on that.