amp icon indicating copy to clipboard operation
amp copied to clipboard

Typescript Syntax highlighting by default

Open bucktower opened this issue 5 years ago • 7 comments

It would be very helpful to include Typescript syntax highlighting by default, given its increasing interest over the past couple years. Here's a ts.sublime-syntax that should work fine for the mean time for everyone else.

bucktower avatar Nov 24 '18 00:11 bucktower

Thanks for this! We're currently leaning on the syntect crate for built-in syntaxes; I think it's worth owning that responsibility in Amp so that we can include these without having to upstream them into that crate. 🙂

jmacdonald avatar Nov 24 '18 03:11 jmacdonald

@bucktower mind adding that to the wiki as well? https://github.com/jmacdonald/amp/wiki

jedahan avatar Nov 24 '18 06:11 jedahan

@jmacdonald How about going like composition and not including specific syntax highlighting to keep amp fast and minimal?

al3rez avatar Nov 24 '18 11:11 al3rez

@jedahan I have some (like Coffeescript, Elixir, Dockerfile, Vue, etc) that I converted from .tmLanguage to .sublime-syntax but I don't know if I can share them on the wiki without attribution etc (I did not trace it and I don'r remember)

pbgc avatar Nov 24 '18 16:11 pbgc

@pbgc I think if you make a repository under your name with your conversions, and write a readme that says "I don't remember where I got all of these. If its yours, let me know and I will add attribution" are you comfortable with that?

jedahan avatar Nov 24 '18 17:11 jedahan

@azbshiri I think that'd violate Amp's philosophy of "batteries included"; major languages/formats should work out of the box. :slightly_smiling_face:

jmacdonald avatar Nov 25 '18 02:11 jmacdonald

JSX should be included too !!

melMass avatar Jan 18 '19 18:01 melMass