-
Notifications
You must be signed in to change notification settings - Fork 5
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
usage of sintaxica as sparql editor; #12
base: sparql_to_spin
Are you sure you want to change the base?
usage of sintaxica as sparql editor; #12
Conversation
@damianw27 This is great work. I created and pushed a separate branch (integrating some later changes I had made to the main code) to work on: https://github.com/william-vw/n3-editor-js/tree/pull_12 One problem is that if I copy paste a bunch of SPARQL queries (plain text, no formatting) I get the following (a bunch of whitespace on the left). Also the layout is bunched to the left (this may also be related to my crappy HTML code): Would it be possible to fix these issues? |
Yup, I will have a look at this after I do my job duties. |
I found few problems in styles. Now everything should fit fine. |
@damianw27 The updated editor.js still doesn't work for the editor (nor your online editor) when programmatically setting the editor value. This is the code you used to update the editor value (with a hard-coded query):
If you try it then the query appears, but after clicking in the editor window, it disappears. Feel free try it yourself here: http://n3-editor.herokuapp.com/n3/editor/sspin3.html Also, when directly setting the query after loading the page nothing happens; if I delay this operation for two seconds, then the query does appear (but with the same "disappearing act" as before). Is there some specific onload event I should be listening for? (As I said, I already wait for the page onload event.) |
In my case, the following code works just fine
Is there any way to test the app with hot reload? |
You can clone the latest repo (sparql_to_n3 branch) and run See lines 296-300 for my efforts on updating the editor, as per the code you had previously shared (and trying the aforementioned delay). |
@william-vw with usage of |
@damianw27 if you can show me an updated PR with functioning code then I will integrate into the editor next week |
No description provided.