Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(781)

Issue 9070044: Fixes #1175019 - Set YUI to not load external CSS. (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
11 years, 5 months ago by huwshimi
Modified:
11 years, 5 months ago
Reviewers:
mp+162685
Visibility:
Public.

Description

Fixes #1175019 - Set YUI to not load external CSS. Set the global YUI config to not load CSS from the YUI CDN. https://code.launchpad.net/~huwshimi/juju-gui/yui-css-loading-1175019/+merge/162685 (do not edit description out of merge proposal)

Patch Set 1 #

Patch Set 2 : Fixes #1175019 - Set YUI to not load external CSS. #

Unified diffs Side-by-side diffs Delta from patch set Stats (+8 lines, -0 lines) Patch
A [revision details] View 1 chunk +2 lines, -0 lines 0 comments Download
M app/modules-debug.js View 1 chunk +3 lines, -0 lines 0 comments Download
M app/modules-prod.js View 1 chunk +3 lines, -0 lines 0 comments Download

Messages

Total messages: 4
huwshimi
Please take a look.
11 years, 5 months ago (2013-05-07 00:22:45 UTC) #1
j.c.sackett
LGTM. Thanks.
11 years, 5 months ago (2013-05-07 04:47:58 UTC) #2
rharding
LGTM please make sure to note a fixes #xxx in the commit message when you ...
11 years, 5 months ago (2013-05-07 15:19:21 UTC) #3
huwshimi
11 years, 5 months ago (2013-05-07 16:26:35 UTC) #4
*** Submitted:

Fixes #1175019 - Set YUI to not load external CSS.

Set the global YUI config to not load CSS from the YUI CDN.

R=j.c.sackett, rharding
CC=
https://codereview.appspot.com/9070044
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld f62528b