Merge lp:~daker/developer-ubuntu-com/fix.1056040-part1 into lp:developer-ubuntu-com/wordpress

Proposed by Adnane Belmadiaf
Status: Merged
Approved by: Michael Hall
Approved revision: 88
Merged at revision: 91
Proposed branch: lp:~daker/developer-ubuntu-com/fix.1056040-part1
Merge into: lp:developer-ubuntu-com/wordpress
Diff against target: 45 lines (+6/-8)
2 files modified
footer.php (+2/-2)
header.php (+4/-6)
To merge this branch: bzr merge lp:~daker/developer-ubuntu-com/fix.1056040-part1
Reviewer Review Type Date Requested Status
Anthony Dillon Approve
Review via email: mp+204675@code.launchpad.net

Commit message

Initial support for SSL content

To post a comment you must log in.
Revision history for this message
Anthony Dillon (ya-bo-ng) wrote :

LGTM, thanks for fixing!

review: Approve
Revision history for this message
Anthony Dillon (ya-bo-ng) wrote :

We also need to update get_template_directory_uri() to return without a protocol.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'footer.php'
2--- footer.php 2013-09-20 12:18:59 +0000
3+++ footer.php 2014-02-04 13:10:09 +0000
4@@ -71,8 +71,8 @@
5 </div>
6 </footer>
7
8-<script type="text/javascript" src="https://ajax.googleapis.com/ajax/libs/jquery/1.6.2/jquery.min.js"></script>
9-<script type="text/javascript" src="https://ajax.googleapis.com/ajax/libs/jqueryui/1.8.14/jquery-ui.min.js"></script>
10+<script type="text/javascript" src="//ajax.googleapis.com/ajax/libs/jquery/1.6.2/jquery.min.js"></script>
11+<script type="text/javascript" src="//ajax.googleapis.com/ajax/libs/jqueryui/1.8.14/jquery-ui.min.js"></script>
12 <script type="text/javascript" src="<?php echo get_template_directory_uri(); ?>/js/jquery.tweet.js"></script>
13 <script type="text/javascript" src="<?php echo get_template_directory_uri(); ?>/js/jquery.slider.js"></script>
14 <script type="text/javascript" src="<?php echo get_template_directory_uri(); ?>/js/base.js"></script>
15
16=== modified file 'header.php'
17--- header.php 2013-09-20 12:18:59 +0000
18+++ header.php 2014-02-04 13:10:09 +0000
19@@ -7,12 +7,12 @@
20
21 <title><?php wp_title('|', true, 'right'); ?><?php bloginfo('name'); ?></title>
22
23-<link rel="stylesheet" type="text/css" href="http://fonts.googleapis.com/css?family=Ubuntu:300,400,400italic">
24-<link rel="stylesheet" type="text/css" href="http://fonts.googleapis.com/css?family=Ubuntu+Mono:400">
25+<link rel="stylesheet" type="text/css" href="//fonts.googleapis.com/css?family=Ubuntu:400,300,300italic,400italic,700,700italic|Ubuntu+Mono">
26 <link rel="stylesheet" type="text/css" media="screen" href="<?php echo get_template_directory_uri(); ?>/css/reset.css" />
27 <link rel="stylesheet" type="text/css" media="screen" href="<?php echo get_template_directory_uri(); ?>/css/960.css" />
28 <link rel="stylesheet" type="text/css" media="screen" href="<?php echo get_template_directory_uri(); ?>/css/base.css" />
29 <link rel="stylesheet" type="text/css" media="screen" href="<?php echo get_template_directory_uri(); ?>/css/<?php echo $template_id; ?>.css" />
30+<link rel="stylesheet" type="text/css" media="screen" href="//assets.ubuntu.com/sites/ubuntu/latest/u/css/global.css" />
31
32 <link rel="pingback" href="<?php bloginfo('pingback_url'); ?>" />
33
34@@ -22,10 +22,8 @@
35
36 <?php wp_head(); ?>
37
38-<link href='https://fonts.googleapis.com/css?family=Ubuntu:400,300,300italic,400italic,700,700italic|Ubuntu+Mono' rel='stylesheet' type='text/css' />
39-<script src="http://assets.ubuntu.com/sites/ubuntu/latest/u/js/plugins/yui-min.js"></script>
40-<link rel="stylesheet" type="text/css" media="screen" href="http://assets.ubuntu.com/sites/ubuntu/latest/u/css/global.css" />
41-<script src="http://assets.ubuntu.com/sites/ubuntu/latest/u/js/global.js"></script>
42+<script src="//assets.ubuntu.com/sites/ubuntu/latest/u/js/plugins/yui-min.js"></script>
43+<script src="//assets.ubuntu.com/sites/ubuntu/latest/u/js/global.js"></script>
44
45 </head>
46

Subscribers

People subscribed via source and target branches