-
Notifications
You must be signed in to change notification settings - Fork 320
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
ACE Syntax Highlighting/Editing needs initial one time (and occasional multiple) page refresh to work #148
Comments
I ran into this as well. No idea why it happens. |
The only change is changing the hosting for the ace editor. It's on cdnjs as with our other libraries. Maybe it's too many requests at Once from them? I remember looking at the chrome network tab once and it pulled in a bunch of files. It's a one liner change to go back to us hosting it though, as it's still in the repo. |
Wonder if that is the reason I have to enable font downloads in order to use it but only in dev. Production I don't currently have to do that. |
…ion-4) Applies to OpenUserJS#198 and OpenUserJS#200 **NOTE** * Why is there a `defer` flag for ace? Could be related to OpenUserJS#148 * Currently ignoring `./public/js`
The only reason this happens is if the server it is hosted on is busy/down... since it's been referenced to our local site I haven't seen this at all including on dev which eliminates the localhost origin issue I was seeing. Closing for now... reopen if necessary. If we ever have a CDN this will probably reappear. |
This is a weird issue that wasn't present before current production deployment yesterday.
Syntax highlighting won't activate in Script Source until (at least) one page refresh happens... then it's usually fine for all other opened script sources.
Haven't tested the usual AO compatibility just yet.
The text was updated successfully, but these errors were encountered: