Cannot read property 'top' of undefined

[piwi91]

In the current Bootstrap 3 (dev-master) I get an error on the offset.
Code:

        place: function () {
        var position = 'absolute';
        var offset = this.component ? this.component.offset() : this.$element.offset();
        this.width = this.component ? this.component.outerWidth() : this.$element.outerWidth();
        offset.top = offset.top + this.height;

        var $window = $(window);

[niftylettuce]

please close this issue and see #13 as I've merged/fixed this issue among others and opened a pull request

[Eonasdan]

check out the latest version. Should have address this issue.

[WhiteRavenTechnology]

May be doing something wrong, but I'm still seeing the "top" issue in the latest build....

Can anyone else verify?

👎

[niftylettuce]

were all of my changes pulled in @Eonasdan?

On Wed, Oct 9, 2013 at 1:07 AM, White Raven Technology <
notifications@github.com> wrote:

May be doing something wrong, but I'm still seeing the "top" issue in the
latest build....

Can anyone else verify?

[image: 👎]

—
Reply to this email directly or view it on GitHubhttps://github.com//issues/5#issuecomment-25947084
.

[Eonasdan]

@niftylettuce for the most part. @WhiteRavenTechnology can you post a jsfiddle or a test page or something?

[WhiteRavenTechnology]

First time messing with jsfiddle, so sorry if it's not what you're after.

This illustrates the problem (kinda) -- the first box works fine (barring some minor css/icon issues).

The second example (no icon, activates on focusing on the text box) breaks -- the error is "cannot read property 'top' of undefined"

http://jsfiddle.net/yBjzP/

Lemme know if you need anything else!

[Eonasdan]

@WhiteRavenTechnology check the new minified file I just uploaded. When I tried your example, I did get cannot read property 'top' of undefined with the min version, but if I used the full copy it worked just fine. I used a different compress tool this time and it worked for me.

[WhiteRavenTechnology]

The raw source is working fine for me now, but the minified version gives me this error now: Uncaught TypeError: Cannot read property 'M' of undefined

Completely kills the script.

Will stick with source, but is anyone else seeing this new error?

[Eonasdan]

hmmm... I usually let Asp.net minify my js on the fly. I'm sorry you're still having trouble. Have you tired a bare minimum test page to make sure it's not something else?

[WhiteRavenTechnology]

It's doing it on the fiddle above, too, and that's just jQuery 1.10.1, bootstrap, and your code. :(

[Eonasdan]

github doesn't usually allow hot linking. I'm not at a desk right now, I'll have to play with it later and get you going

[WhiteRavenTechnology]

Here's a fork of the original fiddle loading the full version and working (mostly) correctly -- a few unrelated CSS issues, but the JS works with the full version, not the minified.

No rush on a fix -- I can work with what I've got for now.

http://jsfiddle.net/6AbrV/

[schnie]

I am also seeing Uncaught TypeError: Cannot read property 'M' of undefined