marijnh/CodeMirror2 — pull request #498 by Gozala (2012-04-17 21:50:04+00:00)
Title: Porting sky-edit's ambiance theme.

This ports sky-edits ambiance theme to CodeMirror:

![Screenshot](http://f.cl.ly/items/0P312g0o0U3I210H0C0i/Screen%20Shot%202012-04-11%20at%2017.40.36%20.png "Screenshot")

Also I got few comments / suggests:

- I think in there is some prefix overload. For example is `.CodeMirror-matchingbracket, .CodeMirror-nonmatchingbracket, .CodeMirror-selected .CodeMirror-focused ...` are those prefixes really necessary ? It would be much easier and portable to have rules like `.CodeMirror .matchingbracket` instead.
- Many themes including default one have selectors like `.cm-s-default span.cm-tag` it feels to me that styling does not needs to know about implementation details like `span`.
- There are rules with subtypes: `.cm-string .cm-string-2` and `.cm-variable, .cm-variable-2, .cm-variable-3` It would be nice if it was something like following instead `.cm-variable, .cm-variable.local .cm-variable.global` so that variable styles are inherited by more specific variable types.
- Parens, Braces, commas, semicolons, operations etc.. don't seem to get their `spans` which is unfortunate as I personally prefer having them visually distinct as shown in the screenshot.

If you find comments above reasonable I'd be happy to provide pull requests for those as well.