-
-
Notifications
You must be signed in to change notification settings - Fork 117
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
Verify that a syntax exists before trying to set it #24
Comments
I asked a question in the ST forum about it: http://www.sublimetext.com/forum/viewtopic.php?f=6&t=16470 |
👍 |
The git hub markdown is cool, did't know about it. I will try it tomorrow with the suggested |
If I don't have MarkdownEditing installed I still get the error:
|
Can you try it again? |
Right now no syntax is set at all
|
I think eventually we'll need some way to match Right now we might just need a way to guess the best Markdown syntax installed, perhaps by checking for |
You have to use |
This human name thing would be a big part maintenance, we would have to match "JScript", "JS", "Java Script", "JavaScript"… for only one language. |
I used the wrong word, I just meant that if the browser knows the language we need (either by figuring it out or by asking the CodeMirror), it will provide a string like CodeMirror has a list of the MIMEs we'd need to support anyway: https://github.com/marijnh/CodeMirror/blob/master/mode/javascript/javascript.js#L672 |
Okey but now we are mixing up the issues, advanced syntax detection can only work if we have Cacodaimon/GhostText-for-Chrome#26 finished and this host name based detection is a way espscially for text areas the only element we are currently supporting. Back to the topic:
|
Yeah I only mentioned that
|
Just wanted to report that I am getting the same error message, even with
set. Output of
|
I told crap No extra Markdown installed:
With MarkdownEditing installed:
This should work, there must be a little bug somewhere, I gonna fix this this evening. |
Can someone please check if it works now, if I got an OK I gonna create a new tag. |
I get this error, both in the console and as an
But I can still set it manually, and the path is the same
|
Can you set your default syntax from
to
this should match to the right syntax, since the output from |
Cool! It started working after I restarted ST. The Can we make it so it defaults to Markdown if what they specify is not found? If Markdown is not found either it should default to Plain Text. Errors can appear in the console |
You are right I gonna change it to your suggested behavior. |
Working fine for me now as well. Great job! |
I've had this problem for a while but I just clicked on "Ok" and it didn't appear anymore until I restarted ST. It looks like I don't have that file installed, I just use the MarkdownEditing package, which is pretty good.
The text was updated successfully, but these errors were encountered: