jonathantneal / closest

Return the closest element matching a selector up the DOM tree

Home Page:http://caniuse.com/#feat=element-closest

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

window is not defined

zaruda opened this issue · comments

window is not defined while server-side rendered

surely you need a DOM in order to use this polyfill, what's the use case for it being SS? I'd suggest you don't apply DOM polyfills when rendering server-side

@willtonkin , this polyfill used by another lib which I'm using in my project.
Only one if statement is necessary to handle problems with ssr

ah, I can see that being annoying. The question then is which project is responsable for being SSR friendly in a chain like that. I'd agree then, this could easily be made to be. I guess it's a question then for the maintainer :)

I will do my best to fix this today.

Hi @jonathantneal !
You can check this updates which were made by coreui team today. coreui/coreui-react#53

Sorry to have missed their cut off, @zaruda. I’ll still try to resolve this today.