7

jQuery $ (this) .position (). top does not work in Chrome

 2 years ago
source link: https://www.codesd.com/item/jquery-this-position-top-does-not-work-in-chrome.html
Go to the source link to view the article. You can view the picture content, updated content and better typesetting reading experience. If the link is broken, please click the button below to view the snapshot at that time.

jQuery $ (this) .position (). top does not work in Chrome

advertisements

I was wondering if there would be any reason this piece of code will not work...

top = $(this).find('ul').position().top;

It works in IE, Firefox, and Safari, however when I alert the output in Chrome it says DOM window object... I need an integer. Why would it alert out DOM window object?


Are you sure you're really alerting that top? Recall that top is a pre-existing global variable in browser-hosted JavaScript (it's window.top, the top-level window).

Update: Interesting, Chrome won't let you implicitly overwrite top (which is probably a good thing): Demo. Just declare your variable (always a good idea anyway) within whatever function that code is in (that code is in a function, right?), which will shadow it, and so that will work: Demo. E.g.:

var top = $(this).find('ul').position().top;

It's important to declare your variables, in order to avoid falling prey to the Horror of Implicit Globals. And as you've found in this case, avoiding globals is always best, as Chrome won't even let you use top as a global if you declare it (again to protect window.top).


About Joyk


Aggregate valuable and interesting links.
Joyk means Joy of geeK