HizurosWoWAddOns / FarmHud

Turn your minimap into a hud for farming ore / herb.
GNU General Public License v3.0
5 stars 6 forks source link

[BUG]: mouse enabled warning fix, sexymap coordinate fix and gather ring increased to match new blizzard range (0.6 instead of 0.432) #11

Open tucomel opened 1 year ago

tucomel commented 1 year ago

WoW client

WoW Retail

What happened?

BUG 1 Sexymap - When the mouse is enabled and you try to move the minimap (certer hud), the dragstop event (for an unknown reason) locks the coordinate position of the minimap and in the next reload (or login logout) the sexymap minimap will appear at middle center of screen (exactly where the hud was draw)

BUG 2 - When show farmhud and enable mouse for tooltip, if you disable the farmhud and reenable again the message of mouse is enable stil appear on middle of screen but the mouse isnt enabled (onhide function didnt hide the message, only the mouse tooltip function) . This bug occurs even with the vanila blizzard minimap

FR - now in dragonflight the gather ring isnt 0.432, the gather ring is 0.6 (tested with a lot of my characters)

Lua Error Message?

none of this cause lua erros, is only behavior errors.

Other addons?

sexymap

ValokAsura commented 4 weeks ago

I noticed the gather circle seemed a bit small in TWW so for fun (and because I'd make a terrible programmer) I multiplied the variables "scaledRadius" on 357 and "gcSize" on 480 in FarmHud.lua by 1.3 as a test. This would put it at 0.56, so I'm betting the .6 mentioned above by tucomel is correct based on my brief tests. It boosts the circle size and pushes the cardinal direction letters out to a more comfortable range.