-
Notifications
You must be signed in to change notification settings - Fork 170
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
Complete Example-Time Control is not working #248
Comments
Well it shouldn't do because it is in controls.js. What, specifically, is the error? |
In handlEvent function, there is no case for UI_TIME, so Line 605 is fired |
Ah right, unless something has change in the meantime, you should notice that it still actually works though - it is just outputting an error incorrectly. I guess we never noticed it was outputting the message on the console log. The TIME control doesn't actually need anything adding to the DOM, as you will see from looking in
and then |
Note: Minor fix to merge in when someone gets a chance |
Hi
CompleteExample line 129:
mainTime = ESPUI.addControl(Time, "", "", None, 0, generalCallback);
In controls.js, it'll fire error unknown Event because Time is not recognized.
thanks
The text was updated successfully, but these errors were encountered: