Skip to content
This repository has been archived by the owner on Jun 10, 2020. It is now read-only.

Why do you need containerHeight? #62

Open
giantelk opened this issue Aug 9, 2015 · 12 comments
Open

Why do you need containerHeight? #62

giantelk opened this issue Aug 9, 2015 · 12 comments

Comments

@giantelk
Copy link

giantelk commented Aug 9, 2015

Hi,

Why do you need to know/specify containerHeight? I was playing with the Semantic UI infinite scroll and it detects the bottom of a DOM element i.e. a <div> automatically without specifying the height of the <div> container.

Here's a JSfiddle with Semantic UI's visibility for infinite scroll:
https://jsfiddle.net/4p6d7x86/13/

@garetht
Copy link
Contributor

garetht commented Aug 11, 2015

The code to do this simply has not yet been written.

@oliviertassinari
Copy link
Contributor

👍

1 similar comment
@yordis
Copy link

yordis commented Dec 7, 2015

👍

@quarryman
Copy link

So is it going to be implemented right away?

@josercruz01
Copy link

+1

@garetht
Copy link
Contributor

garetht commented Jan 21, 2016

No, it is not going to be implemented right away, but if you have a pull request I will be happy to look at it. I imagine one of these days I will be given some time at work to look at this.

@carera
Copy link

carera commented Mar 7, 2016

+1

2 similar comments
@zabojad
Copy link

zabojad commented May 5, 2016

+1

@marcusyip
Copy link

+1

@CaptainYarb
Copy link

Is there any update on this issue?

@yordis
Copy link

yordis commented Jun 3, 2016

@Blazedd read the comment of @garetht

@geminiyellow
Copy link

geminiyellow commented Oct 16, 2017

but... now is 2017, Oct. 2 years later. @garetht do you give up this repo?

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

No branches or pull requests