Code review comment for lp:~michael.nelson/lazr-js/491337-error-style-try-2

Revision history for this message
Michael Nelson (michael.nelson) wrote :

Summary
=======

Fixes bug 491337 by ensuring that the minimal error widget has its own module, so that the associated css is loaded dynamically loaded at runtime when the module appears in a .use() statement.

To test:

Check the following url before and after merging (in FF or Chromium etc.):

file:///home/michael/canonical/lazr-js-branches/491337-error-style-try-2/examples/error/minimal_example.html

Before merging the error will be not be styled, appearing at the top-left of the window.

« Back to merge proposal