Fix footer because too intruisive. closes #1938

[ddurieux]

No description provided.

[coveralls]

Coverage decreased (-24.2%) to 7.222% when pulling 134885c on ddurieux:fix-#1938 into baacaeb on fusioninventory:master.

[trasher]

There still is a duplicate ID, they must be unique.

Why not just add the a content in the last cell of the #footer with a <br/>?

[ddurieux]

Because there is a CSS on the id footer

[trasher]

Take a look with:

diff --git a/js/footer.js b/js/footer.js
index b7fe606..ff4d356 100644
--- a/js/footer.js
+++ b/js/footer.js
@@ -1,16 +1,12 @@
-var fifooter = "<div id='footer'> \
-<table width='100%'>  \
-<tr> \
-<td class='right'> \
+var fifooter = "<br/> \
 <a class='copyright' href='http://fusioninventory.org/'> \
 FusionInventory 9.1+1.0 - Copyleft \
 <span style='display:inline-block;transform: rotate(180deg);font-size: 12px;'>&copy;</span> \
 2010-2016 by FusionInventory Team \
-</a> \
-</td> \
-</tr> \
-</table> \
-</div>"
+</a>";

-$(window).bind("load", function() { $("#footer").after(fifooter); })
+$(window).bind("load", function() {
+   $('#footer').css('height', 'auto');
+   $("#footer td.right").append(fifooter);
+});

Does that look ok for you? Do not seem the existing CSS cause any issue.

[ddurieux]

I've fixed with your recommandation ;)

[coveralls]

Coverage decreased (-0.07%) to 31.396% when pulling cb22f97 on ddurieux:fix-#1938 into baacaeb on fusioninventory:master.

[trasher]

Neat ;)

[trasher]

Unit tests are failing, but this is not due to footer changes; so: approved!

[coveralls]

Coverage decreased (-0.0002%) to 31.407% when pulling 492fb9e on ddurieux:fix-#1938 into babd78c on fusioninventory:master.

[coveralls]

Coverage increased (+29.5%) to 60.886% when pulling 87f0992 on ddurieux:fix-#1938 into babd78c on fusioninventory:master.