CodeMirror 4.0 is about to be released. We should give it a try by testing and maybe updating to the release candidate. The major new features in 4.0 are: - Multiple selections (ctrl-click, alt-drag, and API functions to work with them) - Selection undo/redo (ctrl-u to undo selection and alt-u to redo selection in the default keymap) - A set of Sublime Text keybindings (see [4]) - All modules are wrapped in AMD and CommonJS module loader shims, so that they play well with module loaders. When neither is present, the old style (global CodeMirror variable) is used. - A new character measuring subsystem that is faster (especially on long lines) and more robust (no more problematic corner cases in wrapped text). The last point is the most interesting to me and should fix some long standing issues.
https://groups.google.com/forum/#!topic/codemirror/0kbRVUsM_xo
<rdar://problem/16559681>
CodeMirror 4 has been released. We should do this.
Created attachment 229328 [details] [PATCH] Proposed Fix: Update to CodeMirror 4.0.
Comment on attachment 229328 [details] [PATCH] Proposed Fix: Update to CodeMirror 4.0. View in context: https://bugs.webkit.org/attachment.cgi?id=229328&action=review Looking good. Some minor issues. Also need to rebase so the patch can apply on TOT to land. > Source/WebInspectorUI/ChangeLog:219 > * Localizations/en.lproj/localizedStrings.js: > * UserInterface/Models/DOMNode.js: > * UserInterface/Views/DOMNodeDetailsSidebarPanel.js: > - * UserInterface/Views/Main.css: > + * UserInterface/Views/Main.css: > > 2014-03-28 Joseph Pecoraro <pecoraro@apple.com> > You should revert the edits to the old parts of the ChangeLog. > Source/WebInspectorUI/Scripts/update-codemirror-resources.rb:-46 > - mode/less/less.js Need to remove less.js from Main.html. Not removing it from there will break the Production build. You should also test to make sure combining the resources like Production builds do still works. Especially jsmin.py still works on the new codemirror.js. Define COMBINE_INSPECTOR_RESOURCES=YES while building to trigger this. Or hack copy-user-interface-resources.pl to always do it. > Source/WebInspectorUI/UserInterface/Views/ResourceDetailsSidebarPanel.js:383 > - > + This whole file can be reverted.
Created attachment 229337 [details] [PATCH] Proposed Fix: Update to CodeMirror 4.0.
The production build seems to combine the assets correctly with the new CodeMirror.
Comment on attachment 229337 [details] [PATCH] Proposed Fix: Update to CodeMirror 4.0. Clearing flags on attachment: 229337 Committed r167294: <http://trac.webkit.org/changeset/167294>
All reviewed patches have been landed. Closing bug.