On Monday, March 24, 2014 9:58:53 AM UTC-7, Scott Nelson wrote:
> Thanks David.  I'll keep my eye on that issue.
> 
> Debouncing sounds like a good idea for reducing the number of renders.  I'm 
> solving a similar issue in another part of the app by debouncing the channel 
> containing the updates: 
> https://github.com/scttnlsn/jot/blob/master/src/jot/ui.cljs#L141
> 
> Perhaps this isn't even worth worrying about until there's a noticeable 
> performance problem.  Is this why you're on the fence about some sort of 
> "silent" update?


Caveat : I still have no idea what I'm talking about WRT Om/React. 

That said : Why listen to all scroll events and update your app-state with them 
every time?  Is there no way to preserve this information in component local 
state? 

Would a sliding buffered channel help? Perhaps you could ditch the WillMount 
listener and just get the last value put on the channel in DidMount?

I have no idea if that will work, but it seems you're only consuming the latest 
scrollTop value in DidMount...

-- 
Note that posts from new members are moderated - please be patient with your 
first post.
--- 
You received this message because you are subscribed to the Google Groups 
"ClojureScript" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to [email protected].
To post to this group, send email to [email protected].
Visit this group at http://groups.google.com/group/clojurescript.

Reply via email to