Remove styles tags from SitesManager plugin

plugins/PDFReports/templates/index.twig

@@ -35,7 +35,7 @@
     ReportPlugin.reportList = {{ reportsJSON }};
     ReportPlugin.createReportString = "{{ 'PDFReports_CreateReport'|translate }}";
     ReportPlugin.updateReportString = "{{ 'PDFReports_UpdateReport'|translate }}";
-        $(document).ready(function () {
+    $(function () {
         initManagePdf();
     });
 </script>

plugins/SitesManager/SitesManager.php

@@ -57,6 +57,7 @@ class Piwik_SitesManager extends Piwik_Plugin
     {
         $cssFiles = & $notification->getNotificationObject();
 
+        $cssFiles[] = "plugins/SitesManager/stylesheets/SitesManager.css";
         $cssFiles[] = "plugins/Zeitgeist/stylesheets/styles.css";
     }
 

plugins/SitesManager/javascripts/SitesManager.js

@@ -431,3 +431,28 @@ function onClickSiteSearchUseDefault() {
     }
 }
 
+$(function () {
+
+    // when code element is clicked, select the text
+    $('.trackingHelp code').click(function() {
+        // credit where credit is due:
+        //   http://stackoverflow.com/questions/1173194/select-all-div-text-with-single-mouse-click
+        var range;
+        if (document.body.createTextRange) // MSIE
+        {
+            range = document.body.createTextRange();
+            range.moveToElementText(this);
+            range.select();
+        }
+        else if (window.getSelection) // others
+        {
+            range = document.createRange();
+            range.selectNodeContents(this);
+
+            var selection = window.getSelection();
+            selection.removeAllRanges();
+            selection.addRange(range);
+        }
+    })
+      .click();
+});

plugins/SitesManager/stylesheets/SitesManager.css

+.trackingHelp ul {
+    padding-left: 40px;
+    list-style-type: square;
+}
+
+.trackingHelp ul li {
+    margin-bottom: 10px;
+}
+
+.trackingHelp h2 {
+    margin-top: 20px;
+}
+
+.trackingHelp p {
+    text-align: justify;
+}
+
+#editSites {
+    vertical-align: top;
+}
+
+#editSites h4 {
+    font-size: .8em;
+    margin: 1em 0 1em 0;
+    font-weight: bold;
+}
+
+#editSites .entityTable tr td {
+    vertical-align: top;
+    padding-top: 7px;
+}
+
+#editSites .addRowSite:hover,
+#editSites .editableSite:hover,
+#editSites .addsite:hover,
+#editSites .cancel:hover,
+#editSites .deleteSite:hover,
+#editSites .editSite:hover,
+#editSites .updateSite:hover {
+    cursor: pointer;
+}
+
+#editSites .addRowSite a {
+    text-decoration: none;
+}
+
+#editSites .addRowSite {
+    padding: 1em;
+    font-weight: bold;
+}
+
+.ecommerceInactive,
+.sitesearchInactive {
+    color: #666666;
+}
+
+#searchSiteParameters {
+    display: none;
+}
\ No newline at end of file

plugins/SitesManager/templates/_displayJavascriptCode.twig

-<style type="text/css">
-    .trackingHelp ul {
-        padding-left: 40px;
-        list-style-type: square;
-    }
-
-    .trackingHelp ul li {
-        margin-bottom: 10px;
-    }
-
-    .trackingHelp h2 {
-        margin-top: 20px;
-    }
-
-    p {
-        text-align: justify;
-    }
-</style>
-
 <h2>{{ 'SitesManager_TrackingTags'|translate(displaySiteName) }}</h2>
 
 <div class='trackingHelp'>
@@ -34,31 +15,4 @@
     {{ 'CoreAdminHome_JSTrackingIntro5'|translate('<a target="_blank" href="http://piwik.org/docs/javascript-tracking/">','</a>')|raw }}
     <br/><br/>
     {{ 'Installation_JSTracking_EndNote'|translate('<em>','</em>')|raw }}
-</div>
-<script type="text/javascript">
-    $(document).ready(function () {
-        // when code element is clicked, select the text
-        $('code').click(function () {
-            // credit where credit is due:
-            //   http://stackoverflow.com/questions/1173194/select-all-div-text-with-single-mouse-click
-            var range;
-            if (document.body.createTextRange) // MSIE
-            {
-                range = document.body.createTextRange();
-                range.moveToElementText(this);
-                range.select();
-            }
-            else if (window.getSelection) // others
-            {
-                range = document.createRange();
-                range.selectNodeContents(this);
-
-                var selection = window.getSelection();
-                selection.removeAllRanges();
-                selection.addRange(range);
-            }
-        });
-
-        $('code').click();
-    });
-</script>
\ No newline at end of file
+</div>
\ No newline at end of file