<html><head><meta http-equiv="Content-Type" content="text/html charset=windows-1252"><meta http-equiv="Content-Type" content="text/html charset=windows-1252"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;">Hi,<div>that’s great.</div><div>Can you please send a pull request on github. </div><div>see <a href="https://help.github.com/articles/using-pull-requests">https://help.github.com/articles/using-pull-requests</a></div><div><br></div><div>Thanks,</div><div>Georg Ehrke</div><div><br><div><div>On 08 Jan 2014, at 22:38, R Veach <<a href="mailto:rveach02@gmail.com">rveach02@gmail.com</a>> wrote:</div><br class="Apple-interchange-newline"><blockquote type="cite"><div>I originally submitted a feature request for changing always loading file icon images to lazy loading them to save on bandwidth when opening a folder with many pictures inside it.</div>
<div>The original issue can be read here at <a href="https://github.com/owncloud/core/issues/6591">https://github.com/owncloud/core/issues/6591</a> and all my reasoning for the need of this addition.</div>
<div> </div>
<div>I have made small changes to the owncloud's core app "files" that achieves this functionality and am submitting it here so hopefully it can be incorporated.</div>
<div>The main addition is the JQuery Lazy Load library that I found at <a href="http://www.appelsiini.net/projects/lazyload">http://www.appelsiini.net/projects/lazyload</a> , which can be used, as is, with no modifications. The library is released under the MIT License, which I believe it compatible with owncloud's APGL. Correct me if I am wrong.</div>
<div>The main changes to the file listing is replacing the background image style for the icon and putting it into a custom attribute that Lazy Load recognizes and works with.</div>
<div>The only other change is the call to the Lazy Load function in JS to monitor the user scrolling and position in the document.</div>
<div> </div>
<div>I have tested this with owncloud 6.0.0a and found no issues in IE8 and FF26. The images that are not in view don't load when opening a directory (verified with firebug's net panel), the images are loaded correctly when scrolled into view, and cached images still show up rather instantly. I didn't see any problems with applying the changes to the master in git.</div>
<div> </div>
<div>I have attached the patch using "diff -Naur".</div>
<div>"owncloudFilesOld" is the original core app files directory. "owncloudFilesLazy" is the core app files directory with my modifications for adding lazy load. The core app files directory is located in "owncloud\apps\files".</div>
<span><owncloud lazy load patch.txt></span>_______________________________________________<br>Owncloud mailing list<br><a href="mailto:Owncloud@kde.org">Owncloud@kde.org</a><br><a href="https://mail.kde.org/mailman/listinfo/owncloud">https://mail.kde.org/mailman/listinfo/owncloud</a><br></blockquote></div><br></div></body></html>