-
Notifications
You must be signed in to change notification settings - Fork 54
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[Requests / Bugs] A List of Requests and Bugs? for Addon #192
Comments
I believe this was "fixed" cf6e724 AFAICT Era API was returning nil always, instead of Wrath which returned true/falsey as expected.
This plus surrounding commits may have fixed that 506789e |
I'm on latest build, not sure if those commits have released yet. So hopefully will see them working correctly soon. Thanks! |
Got latest release and talent plans look good. Updated the OP to strikeout that. |
If you're setup for it, I'd appreciate extra validation on latest CF Alpha or |
Sure, I will switch to Alpha and do some testing. Thanks! |
Got the following error. Frames still not retaining position, but can't verify further with error
|
I encountered that error only once and couldn't reproduce it. I attributed it to malformed data I had saved during testing but that doesn't seem to be the case. May be related to userdata placements so would only error if a frame was not moved at all by the user Give 07c27ed a go, seems to work in all my variations. |
Ok, tested and no bug errors. Problem is the setpoint? of where the main frame moves to. I keep my main frame in the bottom right of the screen. So when switching profiles, it is maintaining the bottom right setpoint where I have it kept, but does not retain the width, so it now goes all the way across the screen to the left. Also, the scale for the arrow isn't maintained between profiles. Not sure if the scale on the other windows aren't kept as they aren't up right now. Which brings another thought, can we have an option to show all frames for placement always for placement help. |
Active Target is still not maintaining location by profile. And neither is the scale for it. So I am guessing Active Item window is the same. |
No idea what this error is from, turned in a quest and it popped up. Never seen it before changing to alpha code.
|
Indeed, the I was also only saving/restoring positions for enabledFrames. So the opt-in (like Splits) frames wouldn't be properly loaded or saved. Similarly for the arrow, items, targets because those are evaluated after initial loading based on step data. Should also now save the old profile positions before swapping and loading new profile positions. 86bd7d1 Note: You will likely get an error or two and need at least one |
Not sure how that occurred but should at least not error now. You are correct, scale is not currently handled on profile change, but is restored on reload/login. Will need to think about how to do this in a scalable way instead of hacking something together. |
Tested with new character and it would not set the frames to the profile selection. Got the following error
|
Also, doing a profile reset doesn't actually do anything yet |
More updates (locally), getting close to all these transition scenarios. Reset profile is mostly functional but doesn't handle resetting frame positions to default. Similar (re-)design complication as scale. Similarly swapping profiles is still a little wonky because WoW client itself tracks frame positions. Added error handling for SetPoint ( Thanks for the continual feedback and testing. |
Don't mind testing at all, just keep sending them and I will see what I can find. I will be out of town for a few days, so feedback won't be until friday evening most likely. |
I'm on the alpha channel for CF, is that being updated or do I need to grab the files like I have been from here? Or do you have a fork repo I can sync up with? |
CF Alpha would still be correct. Unfortunately I wasn't able to clean up my local changes yet, been very busy the last couple days. Did some minor cleanup and pushed my changes, in ~15m latest CF Alpha should be ready. |
Tested with the latest push (CF 4.5.24-22-ge3d4146) and frames location, scale did not save or did not load from existing profile. Tested with new toon and used an existing good profile. Also created a new profile to test with and still on a new character, it would not move or scale the frames. Edit: Scale was maintained for Arrow and Active frame, no Item frame open on new character, so not sure about that one. Frames though are still not restoring to profile location. |
Iterated with Starlynk over Discord DMs, profile related issues should be resolved and released in 4.5.25 |
Everything that makes sense as a percentage now should be 27f2032 |
Looks good, been busy so just got time to hop in. But found a different bug with itemupgrades, so posting that one in a new issue. |
Side-effect of AH scanning being beta feature. |
Here is a list of requests/bugs
Profile Set Up
- Update existing profiles to include all aspects of frame data- Currently Profiles do not restore frame position data. Always places it on the left middle of the screenStandardize Options Window
- Scale slider is done in two ways instead of using a single format- Some use a Percentage in the text field, some use a number formatTalent Plans- Talent plans will always say it failed to add talent even when it did- Talent plans do not show the current level you are at on the window for suggestion. It is assuming you have already picked upthat talen when you haven't. This could be part of why it says it failed getting the talent as it's expecting the next rank but can't get that yet.The text was updated successfully, but these errors were encountered: