kaoskeya / meteor-admin

26 stars 2 forks source link

minor jQuery issue - scrollTop #18

Open manueltimita opened 9 years ago

manueltimita commented 9 years ago

This bit of code:

$('html, body').animate({  
  scrollTop: $("#kAdminActionCenter").offset().top 
}, 700);  

...throws an error: "Uncaught TypeError: Cannot read property 'top' of undefined". I guess it is because the form template is not yet rendered when the selector #kAdminActionCenter is being passed to jQuery.

Anyway, excellent package! The first I can properly integrate in my app. No unnecessary bloat, does everything you expect it to do, and no dependency on a specific router (which would be overkill anyway when all you want to do is edit/delete documents).

Manuel

manueltimita commented 9 years ago

The above seems to only happen if the package is loaded inside another smart package. With meteor add everything works fine.