search

Flyer53 / jsPanel3

A jQuery Plugin to create highly configurable floating panels, modals, tooltips, hints/notifiers or contextmenus for use in a backend solution and other web applications.
http://v3.jspanel.de/
Other
89 stars 21 forks source link

[BUG] Dragging jsPanel outside top window margin causes JS error #51

Closed temuri416 closed 7 years ago

temuri416 commented 7 years ago

Hi,

  1. Go to http://jspanel.de/
  2. Grab "Hey there!" window and drag it slowly on top of the top window edge (over bookmarks, for example)
  3. JS error is thrown:

image

Browser: latest Chrome.

Thanks!

Flyer53 commented 7 years ago

Hey Temuri, Sorry for the late answer. Currently I'm moving and almost have no time to turn on the PC. Will stay like that for another week or so ...

So please be patient. The error msg is confirmed (did not show in Chrome 60) and I'll see what I can do.

Regards, Stefan

Flyer53 commented 7 years ago

@temuri416 Just took a look at this issue. It happens only on the home page, not on the other pages. And it happens only with the first and last demo panels sliding in/out. So I think it's not an error in the jsPanel script but in my code to create/remove the demo panels. There also is no negative side effect as far as I can see.

temuri416 commented 7 years ago

That's strange. I'm sure the same happens in my dev instance. I'll keep you posted.

temuri416 commented 7 years ago

I've just confirmed it. It crashed exactly as I described above.

Version:

    version: '3.10.0',
    date: '2017-08-19 12:37',
Flyer53 commented 7 years ago

Ok, I could reproduce the error on other pages/browsers as well now. Have to investigate ... But what exactly do you mean with crash? I never had anything like a frozen or terminating browser. Functionality was always normal.

Flyer53 commented 7 years ago

@temuri416 I made a change to a preliminary v3.11. So far I made only a few test on desktop but it seems to eliminate the error. http://jspanel.de/downloads/jsPanel-3.11.0.zip Maybe you can check in your env ...

temuri416 commented 7 years ago

Works great, can't break it :)

Thanks!

Flyer53 commented 7 years ago

Issue solved with v3.11.0 release