LAZR Javascript Library

Code review comment for lp:~thumper/lazr-js/multi-line-editor-goodness

  1. multi-line-editor-goodness
  2. Merge into toolchain
Revision history for this message
Deryck Hodge (deryck) wrote :

<deryck> thumper, so looks good to me. I would add a js test that the icons are re-enabled on error. And some text on the example page to explain that it will error every other attempt would be nice.
<deryck> thumper, r=me with those changes.
<thumper> deryck: ok

review: Approve (code)

« Back to merge proposal