naver / egjs

Javascript components group that brings easiest and fastest way to build a web application in your way.
https://naver.github.io/egjs
Other
937 stars 81 forks source link

Update description for refresh method in eg.visible. #374

Closed happyhj closed 7 years ago

happyhj commented 8 years ago

Description

The refresh method is not need to be called externally anymore when DOM changes. Because it caches the live collection (NodeList from getElementsByClassName) when it calls first time.

sculove commented 8 years ago

@happyhj The Visible supports IE7+ IE8 doesn't support getElementsByClassName method Therefore, The Visible should have refresh method.

taihoon commented 7 years ago

Can not use getElementsByClassName in IE7, so have to find the target element each time. @happyhj Please check and close the issue.

happyhj commented 7 years ago

@kishu Got it. We Still need refresh method for IE8, IE7!