Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

File Manager: Modals in Firefox on long screens are offset #786

Closed
iliajie opened this issue May 20, 2017 · 6 comments
Closed

File Manager: Modals in Firefox on long screens are offset #786

iliajie opened this issue May 20, 2017 · 6 comments

Comments

@iliajie
Copy link
Collaborator

iliajie commented May 20, 2017

Something makes modal windows to be offset, in Firefox, when the long list of files is in the table. Chrome is displaying content correctly.

Looking for the fix but with no luck so far.

Example:

screenshot from 2017-05-20 17-16-08

#782 (comment)

@iliajie iliajie changed the title File Manager: Modals in Firefox on long screens are off File Manager: Modals in Firefox on long screens are offset May 20, 2017
@adiastra
Copy link

adiastra commented May 24, 2017

This only happens if you scroll down slightly (I just did tests) therefore the modal window is using the top of the page not the top of the visible screen as its marker for top. The offset exactly matches how far down to scroll.

jQuery: How to get the page's current screen top position
What should I do with CSS top value differences in Chrome and Firefox?

@iliajie
Copy link
Collaborator Author

iliajie commented May 24, 2017

The point is that it supposed to be done by Bootstrap. Something is going wrong and I can't figure what.

.. offset exactly matches how far down to scroll ..

This is incorrect. It doesn't.

@adiastra
Copy link

adiastra commented May 24, 2017

It's in the CSS it assumes top: 0, when you scroll down that changes.
top

Can you just make it relative to the center of the screen instead?

@iliajie
Copy link
Collaborator Author

iliajie commented May 25, 2017

It's not the solution.

@7starsone
Copy link

please check this
http://output.jsbin.com/tinotavojo

I don't know if can help...because is done on a table with 3 columns
on iframe you should test it

@iliajie
Copy link
Collaborator Author

iliajie commented Sep 17, 2017

This is no longer an issue in 19.00-RC9 that is released within an hour.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants