Skip to content
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

Closed
Martii opened this issue Jun 12, 2014 · 4 comments
Labels
bug You've guessed it... this means a bug is reported.

Comments

@Martii
Copy link
Member

Martii commented Jun 12, 2014

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.

@sizzlemctwizzle
Copy link
Member

I ran into this as well. No idea why it happens.

@Zren
Copy link
Contributor

Zren commented Jun 13, 2014

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.

@Martii
Copy link
Member Author

Martii commented Jun 13, 2014

changing the hosting for the ace editor

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.

@Martii Martii changed the title ACE Syntax Highlighting needs initial one time page refresh to work ACE Syntax Highlighting/Editing needs initial one time page refresh to work Jun 13, 2014
@Martii Martii changed the title ACE Syntax Highlighting/Editing needs initial one time page refresh to work ACE Syntax Highlighting/Editing needs initial ~~one time~~ multiple page refreshes to work Jun 15, 2014
@Martii Martii changed the title ACE Syntax Highlighting/Editing needs initial ~~one time~~ multiple page refreshes to work ACE Syntax Highlighting/Editing needs initial one time (and occasional multiple) page refresh to work Jun 15, 2014
Martii pushed a commit to Martii/OpenUserJS.org that referenced this issue Jun 26, 2014
…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`
@Martii Martii mentioned this issue Jul 27, 2014
@Martii
Copy link
Member Author

Martii commented Aug 20, 2014

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.

@Martii Martii closed this as completed Aug 20, 2014
@Martii Martii removed the needs testing Anyone can add this but it is primarily there for the Assignee indicating that Testers are wanted. label Apr 27, 2015
@OpenUserJS OpenUserJS locked as resolved and limited conversation to collaborators Apr 12, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug You've guessed it... this means a bug is reported.
Development

No branches or pull requests

3 participants