Author Topic: Cannot build secondary Mage Tower  (Read 803 times)

tomreyn

  • MegaGlest Team
  • Airship
  • ********
  • Posts: 2,764
    • View Profile
    • MegaGlest - the free and open source cross platform 3D real-time strategy game
Cannot build secondary Mage Tower
« on: 14 April 2011, 18:08:04 »
Whenever I select an initiate, then click on the Icon to build a secondary mage tower, nothing happens: I am unable to position the new mage tower, and thus unable to build another one.
atibox: Ryzen 1800X (8 cores @3.6GHz), 32 GB RAM, MSI Radeon RX 580 Gaming X 8G, PCI subsystem ID [1462:3417], (Radeon RX 580 chipset, POLARIS10) @3440x1440; latest stable Ubuntu release, (open source) radeon (amdgpu) / mesa video driver
atibox (old): Core2Quad Q9400 (4 cores @2.66GHz), 8 GB RAM, XFX HD-467X-DDF2, PCI subsystem ID [1682:2931], (Radeon HD 4670, RV730 XT) @1680x1050; latest stable Ubuntu release, (open source) radeon / mesa video driver
notebook: HP envy13d020ng
internet access: VDSL2+

· · · How YOU can contribute to MG · Latest development snapshot · How to build yourself · Megapack techtree · Currently hosted MG games · · ·

hailstone

  • Local Moderator
  • Battle Machine
  • ********
  • Posts: 1,568
    • View Profile
Re: Cannot build secondary Mage Tower
« Reply #1 on: 15 April 2011, 05:01:23 »
I was so close. Silnarm reverted the changes when he merged widget_invert to master so it should work now. Selecting another command on the second tier takes you back to the first tier though. I can change the behaviour so you can reselect a command on the same tier but clicking on the active command won't deactivate it (instead you would need to click the cancel button or right click mouse).
Glest Advanced Engine - Admin/Programmer
https://sourceforge.net/projects/glestae/

silnarm

  • Local Moderator
  • Behemoth
  • ********
  • Posts: 1,373
    • View Profile
Re: Cannot build secondary Mage Tower
« Reply #2 on: 15 April 2011, 06:38:54 »
I was so close. Silnarm reverted the changes when he merged widget_invert to master so it should work now.

Yeah, sorry about that, we'd modified the same functions fairly heavily... got messy.

Feel free to re-tidy, the translated Vec2i as ref param you had for the bounds check was nice, my slightly less clean version was to add Widget::isInsideBorders() ... maybe modify it as such.

Glest Advanced Engine - Code Monkey

Timeline | Downloads

 

anything