nikkisaurus / FarmingBar

4 stars 1 forks source link

Count text y-offset not staying after reload or logout[BUG] #11

Closed Tednik closed 3 years ago

Tednik commented 4 years ago

Describe the bug Bar Count text y-offset is not maintaining its position on reload or logout

Client retail

To Reproduce Steps to reproduce the behavior:

  1. Go to bar settings
  2. Click on '....'
  3. Scroll down to count text y-offset change value 4.reload ui or log out then back in count text does not stay where it was set to but value is still what it should be

Expected behavior A clear and concise description of what you expected to happen.

Screenshots https://i.imgur.com/LBcpBXu.jpg count text set before reload ui bars are on the left side

https://i.imgur.com/qONCp9X.jpg count text after reload ui

Additional context Add any other context about the problem here.

niketa-wow commented 4 years ago

I'm not able to reproduce the issue on my end, so you'll have to bear with me. This might be a little hard to troubleshoot.

I notice that in both the before and after, the positions are the same, so I don't think it's a database issue. It's probably something to do with drawing the buttons.

  1. If you reload again, does the incorrect position persist?
  2. If you change the y-offset, then change it back, does it adjust correctly? And if so, upon reloading does it then revert to the messed up setting?
  3. Are you experiencing any lua errors?
  4. Are you using Masque? If so, what skin are you using on Farming Bar?
Tednik commented 4 years ago
  1. yes it does persist
  2. yes if i change it ot click in the number box and hit enter it adjusts correctly then if i log out or reload yes it reverts to the count text being back on top of the bars
  3. no lua errors
  4. yes im using masque, Masque_CB - Elv checked it with Caith as well does the same thing
niketa-wow commented 4 years ago

Sorry it took so long to get back to you. I've had a busy weekend.

I've fixed the issue and you should be seeing an update in the next half hour or so. Thanks for bringing this to my attention!

Tednik commented 4 years ago

No problem thank you for fixing it