frio: highlight - use offset() instead of postion()

diff --git a/view/theme/frio/js/mod_display.js b/view/theme/frio/js/mod_display.js
index 1faacaeb73251fb0661cbbac2df4392a49d09440..c93d2c00665d6b5f62c65357ad1eba1e715491a8 100644 (file)
--- a/view/theme/frio/js/mod_display.js
+++ b/view/theme/frio/js/mod_display.js
@@ -3,9 +3,9 @@
  */
 
 // Catch the GUID from the URL
-var itemID = window.location.pathname.split("/").pop();
+var itemGuid = window.location.pathname.split("/").pop();
 
 $(document).ready(function(){
        // Scroll to the Item by its GUID
-       scrollToItem(itemID);
+       scrollToItem('item-'+itemGuid);
 });

diff --git a/view/theme/frio/js/theme.js b/view/theme/frio/js/theme.js
index f153fc33ff9672bbe6fddf9c9299825de6cd9194..1e93be24c0d90b1047f67b9501eb03d8e127867b 100644 (file)
--- a/view/theme/frio/js/theme.js
+++ b/view/theme/frio/js/theme.js
@@ -553,11 +553,15 @@ function scrollToItem(itemID) {
        var colWhite = {backgroundColor:'#F5F5F5'};
        var colShiny = {backgroundColor:'#FFF176'};
 
+       // Get the Item Position (we need to substract 100 to match
+       // correct position
+       var itemPos = $('#'+itemID).offset().top - 100;
+
        // Scroll to the DIV with the ID (GUID)
        $('html, body').animate({
-               scrollTop: $('#item-'+itemID).position().top
+               scrollTop: itemPos
        }, 400, function() {
                // Highlight post/commenent with ID  (GUID)
-               $('#item-'+itemID).animate(colWhite, 1000).animate(colShiny).animate(colWhite, 2000);
+               $('#'+itemID).animate(colWhite, 1000).animate(colShiny).animate(colWhite, 600);
        });
 }