Merge lp:~stevenk/launchpad/privacy-portlet-colour-change into lp:launchpad

Proposed by Steve Kowalik on 2012-04-30
Status: Merged
Approved by: Steve Kowalik on 2012-05-01
Approved revision: no longer in the source branch.
Merged at revision: 15182
Proposed branch: lp:~stevenk/launchpad/privacy-portlet-colour-change
Merge into: lp:launchpad
Diff against target: 23 lines (+0/-13)
1 file modified
lib/lp/app/javascript/privacy.js (+0/-13)
To merge this branch: bzr merge lp:~stevenk/launchpad/privacy-portlet-colour-change
Reviewer Review Type Date Requested Status
Ian Booth (community) 2012-04-30 Approve on 2012-04-30
Review via email: mp+104188@code.launchpad.net

Commit Message

Drop code to change privacy portlet red when hiding the privacy banner.

Description of the Change

The privacy JS will currently change the background of any privacy portlet it finds to dark red when the hide_privacy_notification() method is called. This is causing issues with information type changing, and the code should no longer be used. In fact, it also looks to have been untested.

To post a comment you must log in.
Ian Booth (wallyworld) wrote :

Looks ok. Collective opinion is that this code dates from when privacu banner could be hidden so can safely go.

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'lib/lp/app/javascript/privacy.js'
2--- lib/lp/app/javascript/privacy.js 2011-09-26 06:30:07 +0000
3+++ lib/lp/app/javascript/privacy.js 2012-04-30 23:42:28 +0000
4@@ -127,19 +127,6 @@
5 fade_out.run();
6 body_space.run();
7 login_space.run();
8-
9- var privacy_portlet = Y.one('.portlet.private');
10- if (privacy_portlet !== null) {
11- var portlet_colour = new Y.Anim({
12- node: privacy_portlet,
13- to: {
14- color: '#fff',
15- backgroundColor:'#8d1f1f'
16- },
17- duration: 0.4
18- });
19- portlet_colour.run();
20- }
21 }
22 }
23 namespace.hide_privacy_notification = hide_privacy_notification;