Git Product home page Git Product logo

Comments (16)

fabien-d avatar fabien-d commented on May 24, 2024

It's weird since the page shouldn't scroll, like you said it's simply a position fix. Sounds like it may not prevent the default event on the button click.

  • Does it happen with all three dialogs, or just prompt?
  • Can you recreate on the example site?
  • Does it append a blank anchor # to URL after button click?

If you could provide screenshots and/or post more info (code sample to recreate) I can try to debug.

from alertify.js.

dschwen avatar dschwen commented on May 24, 2024

It happens with all three dialogs.
Yeah it happens on the example site as well.
It does not append a hash.

I dont know what to show on the screenshot, except for a scrolled-up page ;-).

from alertify.js.

fabien-d avatar fabien-d commented on May 24, 2024

yeah, fair enough... What OS? And can you reproduce on any other browsers (or are other browsers good like FF)?

from alertify.js.

dschwen avatar dschwen commented on May 24, 2024

Ha, sorry, MacOSX 10.7.3. I'll try later at home on Linux and a brand new Chrome. For some reason in my Linux Virtualbox here popping up the dialog freezes up my browsertab.
I also tried Safari and no dialogs show up at all (Version 5.1.5 (7534.55.3))!
As I said FF 17 perfroms perfectly.

from alertify.js.

fabien-d avatar fabien-d commented on May 24, 2024

I think I was able to reproduce, I'll look into it some more shortly and hopefully get this resolved soon.

from alertify.js.

fabien-d avatar fabien-d commented on May 24, 2024

Tied to this #81 - I can reproduce the Safari 5.1.* issue where the dialogs don't show which I believe is what you mean by freezes up (dialog doesn't show, but cover does - meaning you can't click on anything)

Still investigating the jump to top issue.

from alertify.js.

fabien-d avatar fabien-d commented on May 24, 2024

Jump to Top

This is a result of setting focus to the OK button or input focus before the animation is complete. Currently a setTimeout with a 50ms delay is used which resolves issues with certain browsers that weren't setting focus at all. Setting a higher delay, e.g. 1000ms resolved the jump but is really ugly. Looking for a cleaner more reliable approach. (Thinking of using the transitionend instead of setTimeout to set focus)

Dialogs not Showing

There is a parameter of visibility: hidden in the .alertify-hidden class in the core alertify.core.css file. Removing this resolves the dialogs not showing but causes a failure in IE8-. Looking for a solution that will work cross browser.

Thanks for logging the issue - this will help solve major flaws.

from alertify.js.

fabien-d avatar fabien-d commented on May 24, 2024

Issues should be resolved with ^ pull request.

from alertify.js.

dschwen avatar dschwen commented on May 24, 2024

Thanks, just tested it, works well in Chrome and Safari now ! Love your little library. Thanks a bunch again!

from alertify.js.

fabien-d avatar fabien-d commented on May 24, 2024

Awesome. Closing this issue. If anything comes up simply re-open or log a new one. Cheers

from alertify.js.

fizerkhan avatar fizerkhan commented on May 24, 2024

It is still happens to me. How to fix this bug?

from alertify.js.

sangel10 avatar sangel10 commented on May 24, 2024

I am having the scrolling issue on Chrome 33 on OSX 10.7.5 on version 0.3.11, is there any way to manually fix this?

from alertify.js.

PetrKapishev avatar PetrKapishev commented on May 24, 2024

I have this issue in Chrome (Linux) too.
I don't know a nature of this problem, so I had to comment the line 432 (elCallee.focus()) to fix it.

from alertify.js.

sangel10 avatar sangel10 commented on May 24, 2024

PetruhaXI's solution worked for me. I think it might have to do with the fact that I removed the element that called the notification so there was no longer an element to focus on after the call.

from alertify.js.

umair-me avatar umair-me commented on May 24, 2024

PetruhaXI's solution worked for me too.

from alertify.js.

toyrunr avatar toyrunr commented on May 24, 2024

I am having the same problem on Windows 7 in IE11. Page scrolls to top for every alert type displayed. the 432 line is no longer what it used to be. I played around with various "focus" lines of code, to no avail. Thanks

from alertify.js.

Related Issues (20)

Recommend Projects

  • React photo React

    A declarative, efficient, and flexible JavaScript library for building user interfaces.

  • Vue.js photo Vue.js

    🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.

  • Typescript photo Typescript

    TypeScript is a superset of JavaScript that compiles to clean JavaScript output.

  • TensorFlow photo TensorFlow

    An Open Source Machine Learning Framework for Everyone

  • Django photo Django

    The Web framework for perfectionists with deadlines.

  • D3 photo D3

    Bring data to life with SVG, Canvas and HTML. 📊📈🎉

Recommend Topics

  • javascript

    JavaScript (JS) is a lightweight interpreted programming language with first-class functions.

  • web

    Some thing interesting about web. New door for the world.

  • server

    A server is a program made to process requests and deliver data to clients.

  • Machine learning

    Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.

  • Game

    Some thing interesting about game, make everyone happy.

Recommend Org

  • Facebook photo Facebook

    We are working to build community through open source technology. NB: members must have two-factor auth.

  • Microsoft photo Microsoft

    Open source projects and samples from Microsoft.

  • Google photo Google

    Google ❤️ Open Source for everyone.

  • D3 photo D3

    Data-Driven Documents codes.