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

Firefox Large Height Issue #9

Open
chandlerkent opened this issue Jul 11, 2016 · 2 comments
Open

Firefox Large Height Issue #9

chandlerkent opened this issue Jul 11, 2016 · 2 comments

Comments

@chandlerkent
Copy link

In reference to:

Firefox has a nasty bug (https://bugzilla.mozilla.org/show_bug.cgi?id=373875) that breaks any attempt of assigning big numerical values to css properties. Since the virtual list does exactly that to give the illusion of a very big list without actually loading the components, you might run into that bug for very big lists. Unfortunately, I haven't found a way to work around it yet.

If I understand the problem correctly, in the past I've worked around this by having multiple "spacer" elements in the DOM which are of the maximum size. As long as no single element is bigger than the maximum height, this seems to work. So instead of setting the container element to 1,000,000 px (for example), you have multiple spacer children which are each 10,000 px for example. The spacer elements and the actual elements that are shown will together add up to 1,000,000 px. Hope that makes some sense!

IE also has the same issue btw (not sure about Edge).

@tbranyen
Copy link

tbranyen commented Aug 3, 2016

Browsers still tend to do weird things when your container has elements that exceed a certain height. I'd rather these tools "do the right thing" rather than implement quirks to get around issues with the rendering engine.

@chandlerkent
Copy link
Author

@tbranyen thanks for the feedback.

What would the right thing be in this scenario? Right now this library will break when you attempt to show a certain # of items (or you have items with large heights).

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

No branches or pull requests

2 participants