Closed ajsmit24 closed 6 years ago
Woohoo thanks Andrew!!
Let Gwen and Sufy and I know when this goes live :-)
On Wed, Jun 27, 2018 at 4:22 PM ajsmit24 notifications@github.com wrote:
Andrew's performance fixes Implemented in a front-end redesign to increase performance by reducing jQuery look up reliance. Also known as not creating 100,000 divs.
You can view, comment on, or merge this pull request online at:
https://github.com/CMU-CREATE-Lab/esdr/pull/5 Commit Summary
- front-end redesign to increase performance
File Changes
- M public/browse/index.html https://github.com/CMU-CREATE-Lab/esdr/pull/5/files#diff-0 (765)
Patch Links:
- https://github.com/CMU-CREATE-Lab/esdr/pull/5.patch
- https://github.com/CMU-CREATE-Lab/esdr/pull/5.diff
— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/CMU-CREATE-Lab/esdr/pull/5, or mute the thread https://github.com/notifications/unsubscribe-auth/AAT082eWbUngMs7FCxB0m74k7jwb2C2fks5uA-mjgaJpZM4U6VAv .
Yeah, I can vouch for the huge speed increase. Thanks again Andrew!!
On 6/27/2018 4:24 PM, rsargent wrote:
Woohoo thanks Andrew!!
Let Gwen and Sufy and I know when this goes live :-)
On Wed, Jun 27, 2018 at 4:22 PM ajsmit24 notifications@github.com wrote:
Andrew's performance fixes Implemented in a front-end redesign to increase performance by reducing jQuery look up reliance. Also known as not creating 100,000 divs.
You can view, comment on, or merge this pull request online at:
https://github.com/CMU-CREATE-Lab/esdr/pull/5 Commit Summary
- front-end redesign to increase performance
File Changes
- M public/browse/index.html https://github.com/CMU-CREATE-Lab/esdr/pull/5/files#diff-0 (765)
Patch Links:
- https://github.com/CMU-CREATE-Lab/esdr/pull/5.patch
- https://github.com/CMU-CREATE-Lab/esdr/pull/5.diff
— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/CMU-CREATE-Lab/esdr/pull/5, or mute the thread
— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/CMU-CREATE-Lab/esdr/pull/5#issuecomment-400816910, or mute the thread https://github.com/notifications/unsubscribe-auth/AA4984_ZTh-RXHiFC2wOi49gNBG2xhXcks5uA-oTgaJpZM4U6VAv.
Let's see if I can find the right emoji to express how I feel right now:
🙇♀️ 🙇♂️🙇♀️ 🙇♂️🙇♀️ 🙇♂️🙇♀️ 🙇♂️🙇♀️ 🙇♂️🙇♀️ 🙇♂️🙇♀️ 🙇♂️
THANK YOU SO MUCH!!! This has been bugging me for three years now and I'm so happy that it has been resolved. I've learned so much since I worked on that site and looking back, I definitely would not have made the same decisions. I was hoping to make an attempt at fixing the site this summer but if Andrew's going to do it, I'd be happy to sit back and watch :)
Andrew's performance fixes Implemented in a front-end redesign to increase performance by reducing jQuery look up reliance. Also known as not creating 100,000 divs.