Merge lp:~daker/ubuntu-html5-theme/fix.1241215 into lp:~ubuntu-html5-theme-devs/ubuntu-html5-theme/ambiance

Proposed by Adnane Belmadiaf
Status: Merged
Approved by: Kyle Nitzsche
Approved revision: 90
Merged at revision: 88
Proposed branch: lp:~daker/ubuntu-html5-theme/fix.1241215
Merge into: lp:~ubuntu-html5-theme-devs/ubuntu-html5-theme/ambiance
Diff against target: 12 lines (+1/-1)
1 file modified
0.1/ambiance/js/list.js (+1/-1)
To merge this branch: bzr merge lp:~daker/ubuntu-html5-theme/fix.1241215
Reviewer Review Type Date Requested Status
PS Jenkins bot continuous-integration Approve
Kyle Nitzsche Approve
Adnane Belmadiaf Needs Resubmitting
Review via email: mp+191899@code.launchpad.net

Commit message

s/parentNode/parendNode
Spaces instead of tabs

To post a comment you must log in.
Revision history for this message
Kyle Nitzsche (knitzsche) wrote :

This MR shows changes on almost every line since it also appears to contain the fix for https://bugs.launchpad.net/ubuntu-html5-theme/+bug/1240682 (some js files use tabs, others spaces for indentation).

Adnane, would you prefer to modify this branch to fix only the List.remove() bug?

89. By Adnane Belmadiaf

Reverted tab to space change

90. By Adnane Belmadiaf

Refix

Revision history for this message
Adnane Belmadiaf (daker) wrote :

Done.

review: Needs Resubmitting
Revision history for this message
Adnane Belmadiaf (daker) wrote :

Can you please top-approve this first so i can fix this MR https://code.launchpad.net/~daker/ubuntu-html5-theme/fix.1240682/+merge/191900 ?

Revision history for this message
Kyle Nitzsche (knitzsche) wrote :

thanks

review: Approve
Revision history for this message
PS Jenkins bot (ps-jenkins) :
review: Approve (continuous-integration)

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file '0.1/ambiance/js/list.js'
--- 0.1/ambiance/js/list.js 2013-07-17 15:27:43 +0000
+++ 0.1/ambiance/js/list.js 2013-10-21 14:03:41 +0000
@@ -130,7 +130,7 @@
130 remove: function (index) {130 remove: function (index) {
131 var item = this.at(index);131 var item = this.at(index);
132 if (item) {132 if (item) {
133 item.parendNode.removeChild(item);133 item.parentNode.removeChild(item);
134 }134 }
135 },135 },
136 /*136 /*

Subscribers

People subscribed via source and target branches