Skip to content
This repository has been archived by the owner on Sep 6, 2021. It is now read-only.

Add Dart mode #10308

Merged
merged 1 commit into from
Jan 13, 2015
Merged

Add Dart mode #10308

merged 1 commit into from
Jan 13, 2015

Conversation

marcelgerber
Copy link
Contributor

The new CM version (not yet in Brackets) has a Dart mode implemented.
For #6087.

@peterflynn
Copy link
Member

@marcelgerber Is this mergable before the new CM lands? It seems like LanguageManager will emit console errors (at the least) since the mode doesn't exist yet.

@marcelgerber
Copy link
Contributor Author

I haven't tested it, but it's probably best to wait for the CM merge.
I just didn't want to forget this as it was an often-wished feature and it was easy to do.

@marcelgerber
Copy link
Contributor Author

@peterflynn Time for review now that we have latest CM.

@peterflynn
Copy link
Member

Cool, looks good! I find it weird that the mimetype starts with application/ instead of text/ like most other CodeMirror modes, but that's how it's defined upstream... so it's not our problem.

peterflynn added a commit that referenced this pull request Jan 13, 2015
@peterflynn peterflynn merged commit 89ffbb9 into adobe:master Jan 13, 2015
@marcelgerber marcelgerber deleted the dart-mode branch January 14, 2015 13:05
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants