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

Manual: updated [email protected] #25454

Merged
merged 2 commits into from
Feb 7, 2023
Merged

Conversation

linbingquan
Copy link
Contributor

@linbingquan linbingquan commented Feb 7, 2023

Related issue: #XXXX

Description

Removed manual/3rdparty/monaco-editor dir.

Updated monaco-editor lastest version with https://unpkg.com/[email protected]/min/vs link.

I tested in my local computer, it can work, if has any problem, please tell me.

@mrdoob mrdoob added this to the r150 milestone Feb 7, 2023
@mrdoob mrdoob merged commit 0bf38ce into mrdoob:dev Feb 7, 2023
@linbingquan linbingquan deleted the dev-monaco-editor branch February 7, 2023 02:59
@epreston
Copy link
Contributor

epreston commented Feb 7, 2023

So manual does not work offline, is broken, and still links the following in some files.

<script src="/manual/3rdparty/monaco-editor/min/vs/loader.js"></script>

@epreston
Copy link
Contributor

epreston commented Feb 7, 2023

Two options:

  • fix all references to use the package manager
  • update this one by downloading the most recent update and replacing the files

@epreston
Copy link
Contributor

epreston commented Feb 7, 2023

To get better tests, open the chrome dev tools, in the network tab click "disable cache". Then click on each link to see any 404 errors. You can view the console for javascript errors at the same time by pressing esc key. Also, before you test, do a quick search for references in the code. You using vscode ?

@linbingquan
Copy link
Contributor Author

So manual does not work offline, is broken, and still links the following in some files.

<script src="/manual/3rdparty/monaco-editor/min/vs/loader.js"></script>

Thank you for your reply.

Is that right?

--- a/manual/examples/resources/editor.html
+++ b/manual/examples/resources/editor.html
@@ -277,7 +277,7 @@ button:focus {
   </div>
 </div>
 </body>
-<script src="/manual/3rdparty/monaco-editor/min/vs/loader.js"></script>
+<script src="https://unpkg.com/[email protected]/min/vs/loader.js"></script>
 <script src="editor-settings.js"></script>
 <script src="editor.js"></script>

@linbingquan
Copy link
Contributor Author

To get better tests, open the chrome dev tools, in the network tab click "disable cache". Then click on each link to see any 404 errors. You can view the console for javascript errors at the same time by pressing esc key. Also, before you test, do a quick search for references in the code. You using vscode ?

Thank you for your suggestion, I tested code it can be work in browser(with cache), I think it is no problem, my mistake...

@epreston
Copy link
Contributor

epreston commented Feb 7, 2023

Happy to help.

Are you sure everyone is ok with the manual being something that only works when connected to the internet ?

I like reducing the size of the project so I have no issue. I hope this doesn't disrupt anyone elses workflow.

@mrdoob
Copy link
Owner

mrdoob commented Feb 7, 2023

This one seems a bit more complicated...
I think we can give it a try and see if anyone complains.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants