-
Notifications
You must be signed in to change notification settings - Fork 47
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
Flux Editor throws errors when used in a loop that changes order #806
Comments
@joshhanley do you have an update on this one, or an ETA for when it might be fixed? |
@mikecchops yep, I’ve fixed it! Hopefully Caleb should review it next week. 😊 |
@joshhanley great thanks Josh! I look forward to the update. |
@joshhanley Hi Josh - what's the update on this one? Has it been fixed on either of the latest releases? |
@mikecchops Caleb has reviewed it 2 days ago, but wasn't happy with the solution, so it needs reworking. That being said, while the error is being thrown, the editor component was still functional in my testing, so it should still all work fine until we get the error cleaned up. |
@joshhanley thanks for the update. Yes the editor works, but the linked issue of the content changing on sort is still an issue. |
Currently if you use the Editor component in a loop and change the order of the loop or prepend to it, the following error gets thrown multiple times. But if you just append to the loop, it doesn't throw any errors. Interestingly even though the error gets thrown, all editors still seem to function as expected.
Open the below Volt component in your browser and open the console, then either click
Append Item
which should work fine, or clickPrepend Item
which throws the error.The text was updated successfully, but these errors were encountered: