Bug 130019 – Web Inspector: Consider updating to CodeMirror 4.0

• Status: RESOLVED
• Resolution: FIXED
• Component: Web Inspector

[Timothy Hatcher]

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.

[Timothy Hatcher]

https://groups.google.com/forum/#!topic/codemirror/0kbRVUsM_xo

[Radar WebKit Bug Importer]

<rdar://problem/16559681>

[Timothy Hatcher]

CodeMirror 4 has been released. We should do this.

[Jonathan Wells]

Created attachment 229328 [details]
[PATCH] Proposed Fix: Update to CodeMirror 4.0.

[Timothy Hatcher]

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  <[e-mail]>
>  

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.

[Jonathan Wells]

Created attachment 229337 [details]
[PATCH] Proposed Fix: Update to CodeMirror 4.0.

[Jonathan Wells]

The production build seems to combine the assets correctly with the new CodeMirror.

[WebKit Commit Bot]

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>

[WebKit Commit Bot]

All reviewed patches have been landed.  Closing bug.