Merge lp:~whosdaz/ubuntu-qa-website/dev into lp:ubuntu-qa-website

Proposed by Matthew Allen
Status: Merged
Merged at revision: 424
Proposed branch: lp:~whosdaz/ubuntu-qa-website/dev
Merge into: lp:ubuntu-qa-website
Diff against target: 44 lines (+1/-18)
3 files modified
modules/qatracker/user/qatracker.user.subscriptions.php (+1/-0)
modules/qawebsite/misc/qawebsite_fix1533472.css (+0/-3)
modules/qawebsite/qawebsite.module (+0/-15)
To merge this branch: bzr merge lp:~whosdaz/ubuntu-qa-website/dev
Reviewer Review Type Date Requested Status
Nicholas Skaggs (community) Approve
Review via email: mp+284115@code.launchpad.net

Description of the change

Commit message:

Removed the CSS Override file, that seems to be unneccesary as when disabled in chrome developer tools the issue it was added to fix is still resolved when the changes are reverted

To post a comment you must log in.
Revision history for this message
Nicholas Skaggs (nskaggs) wrote :

Committing to test this out -- The hack shouldn't stay in prod, but if it fails, we'll rollback.

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'modules/qatracker/user/qatracker.user.subscriptions.php'
--- modules/qatracker/user/qatracker.user.subscriptions.php 2012-11-19 20:35:32 +0000
+++ modules/qatracker/user/qatracker.user.subscriptions.php 2016-01-27 13:23:20 +0000
@@ -71,6 +71,7 @@
71 );71 );
7272
73 return $items;73 return $items;
74
74}75}
7576
7677
7778
=== removed file 'modules/qawebsite/misc/qawebsite_fix1533472.css'
--- modules/qawebsite/misc/qawebsite_fix1533472.css 2016-01-14 17:51:38 +0000
+++ modules/qawebsite/misc/qawebsite_fix1533472.css 1970-01-01 00:00:00 +0000
@@ -1,3 +0,0 @@
1tbody {
2 color: #000;
3}
4\ No newline at end of file0\ No newline at end of file
51
=== modified file 'modules/qawebsite/qawebsite.module'
--- modules/qawebsite/qawebsite.module 2016-01-20 21:13:07 +0000
+++ modules/qawebsite/qawebsite.module 2016-01-27 13:23:20 +0000
@@ -39,19 +39,4 @@
39 drupal_add_html_head($redirect, 'redirect');39 drupal_add_html_head($redirect, 'redirect');
40}40}
4141
42
43// Add custom css to ovveride coloring of tables - this is a workaround and shouldn't be a long term fix
44// Fixes https://bugs.launchpad.net/ubuntu-qa-website/+bug/1533472
45
46$cssoverride = array(
47 '#tag' => 'link', // The #tag is the html tag - <link />
48 '#attributes' => array( // Set up an array of attributes inside the tag
49 'rel' => 'stylesheet',
50 'type' => 'text/css',
51 'href' => '/modules/qawebsite/misc/qawebsite_fix1533472.css',
52 ),
53);
54
55drupal_add_html_head($cssoverride, 'cssoverride');
56
57?>42?>

Subscribers

People subscribed via source and target branches