https://redmine.gc.cuny.edu/https://redmine.gc.cuny.edu/favicon.ico2013-09-03T15:34:22ZCUNY Graduate Center - Project Tracking SystemCUNY Academic Commons - Bug #2772: Fellowship website layout bughttps://redmine.gc.cuny.edu/issues/2772?journal_id=128372013-09-03T15:34:22ZDominic Gigliohumanshell@gmail.com
<ul></ul><p>Hey Erin!</p>
<p>Summer was fine, went by WAY too fast. Hope your's was great.</p>
<p>Just started looking at the site, it appears to be a javascript issue at first glance. The slider appears to be broken and is outputting all the images instead of hiding them and then sliding them.</p>
<p>It looks broken in Safari but appears to work in Chrome. What browser are you using?</p> CUNY Academic Commons - Bug #2772: Fellowship website layout bughttps://redmine.gc.cuny.edu/issues/2772?journal_id=128382013-09-03T15:38:27Zerin glasserin.glass@gmail.com
<ul></ul><p>Agree about the speedy summer!</p>
<p>The site looks broken on Safari and Firefox. I don't have Chrome installed on my office computer -- but I will put in a request!</p>
<p>Thanks!</p> CUNY Academic Commons - Bug #2772: Fellowship website layout bughttps://redmine.gc.cuny.edu/issues/2772?journal_id=128392013-09-03T16:00:25ZMatt Goldmattgold@gmail.com
<ul></ul><p>please also test in IE, which is what most computers at the GC have set as default. Many thanks.</p> CUNY Academic Commons - Bug #2772: Fellowship website layout bughttps://redmine.gc.cuny.edu/issues/2772?journal_id=128402013-09-03T16:01:21ZDominic Gigliohumanshell@gmail.com
<ul></ul><p>OK, let me dig around and see what's going on here.</p> CUNY Academic Commons - Bug #2772: Fellowship website layout bughttps://redmine.gc.cuny.edu/issues/2772?journal_id=128412013-09-03T16:12:59ZMatt Goldmattgold@gmail.com
<ul><li><strong>Status</strong> changed from <i>New</i> to <i>Assigned</i></li></ul> CUNY Academic Commons - Bug #2772: Fellowship website layout bughttps://redmine.gc.cuny.edu/issues/2772?journal_id=128522013-09-06T14:29:42Zerin glasserin.glass@gmail.com
<ul></ul><p>Hi Dom,</p>
<p>Sorry to check in so early, but the project manager of the site is eager to hear about when the homepage might be fixed. Let me know if you think it will be next week or later and I can send her an update.</p>
<p>Many, many thanks!</p>
<p>Erin</p> CUNY Academic Commons - Bug #2772: Fellowship website layout bughttps://redmine.gc.cuny.edu/issues/2772?journal_id=128542013-09-06T15:22:48ZKeith Miyakekmiyake@gc.cuny.edu
<ul></ul><p>the problem is that the theme is trying to dynamically load a pretty outdated version of jquery after the main WP jquery call:<br /><script type="text/javascript" src="http://ajax.googleapis.com/ajax/libs/jquery/1.5.2/jquery.min.js?ver=1.5.2"></script></p>
<p>this breaks the slider and any other jquery functionality.</p>
<p>given the place in the header where this call is made, I have a feeling it's in the theme.</p>
<p>I just did a fresh install of luna on a blank site and saw the same header include so I'm almost positive that's the problem.</p>
<p>the easiest fix (that Dom or another admin has to do) is to just remove that line from the header.php or the static include in the theme's functions.php. Since they didn't enqueue the script I'm putting money on the header.php.</p>
<p>Removing the line in Chrome's Dev Tools got jquery up and working again.</p> CUNY Academic Commons - Bug #2772: Fellowship website layout bughttps://redmine.gc.cuny.edu/issues/2772?journal_id=128562013-09-06T18:55:35ZDominic Gigliohumanshell@gmail.com
<ul></ul><p>Keith,</p>
<p>Thanks for the heads up! I really appreciate it.</p>
<p>Erin,</p>
<p>It <strong>definitely</strong> will not be next week. Now that Keith has pointed out what most likely is the exact cause I should have it fixed tonight or tomorrow.</p> CUNY Academic Commons - Bug #2772: Fellowship website layout bughttps://redmine.gc.cuny.edu/issues/2772?journal_id=128832013-09-09T17:21:44ZDominic Gigliohumanshell@gmail.com
<ul></ul><p>Hey Erin,</p>
<p>Sorry, been tied up with the startup I'm working on.</p>
<p>I'm looking at the theme right now and Keith is almost correct. Luna is enqueueing the old jQuery file in its <code>includes/custom/custom-functions.php</code> file. As soon as I figure out the correct code to prevent this from being loaded in the header I will push up the fix and it will be working again after the next release (Wednesday).</p>
<p>I'll update again here as soon as I push up the fix.</p> CUNY Academic Commons - Bug #2772: Fellowship website layout bughttps://redmine.gc.cuny.edu/issues/2772?journal_id=128882013-09-09T18:19:38ZKeith Miyakekmiyake@gc.cuny.edu
<ul></ul><p>Dom, if you attach the custom-functions.php file I can probably get you a quick fix. I can't access the theme files directly since it's a premium theme so maybe you can attach it on here.</p> CUNY Academic Commons - Bug #2772: Fellowship website layout bughttps://redmine.gc.cuny.edu/issues/2772?journal_id=128922013-09-09T18:29:06ZDominic Gigliohumanshell@gmail.com
<ul></ul><p>No need, Keith. I got it figured out. I created a function in the Fellowship's child <code>functions.php</code> and attached it to the <code>wp_print_sctipts</code> action. It effectively removes the registration of the offending jQuery version that's being loaded from Google, fixing the current issue with the slider on the homepage. I really didn't want to edit the Luna parent directly. I would have if there were no other options, but staying inside the child theme is always the best bet.</p>
<p>I am waiting on an answer from another issue right now, and as soon as I get that answer I'll be able to push this fix up for the release on Wed.</p>
<p>Thank you again for looking into this issue. Your feedback helped a lot.</p> CUNY Academic Commons - Bug #2772: Fellowship website layout bughttps://redmine.gc.cuny.edu/issues/2772?journal_id=128932013-09-09T19:25:09ZDominic Gigliohumanshell@gmail.com
<ul><li><strong>Target version</strong> set to <i>1.5.2</i></li></ul> CUNY Academic Commons - Bug #2772: Fellowship website layout bughttps://redmine.gc.cuny.edu/issues/2772?journal_id=129432013-09-11T16:24:52ZDominic Gigliohumanshell@gmail.com
<ul></ul><p>The updates discussed above have been pushed to our repo, so they'll be included in today's release. I will verify the fix after the release and update here to let everyone know whether or not this issue can be closed.</p> CUNY Academic Commons - Bug #2772: Fellowship website layout bughttps://redmine.gc.cuny.edu/issues/2772?journal_id=129792013-09-12T00:40:57ZBoone Gorgesboone@gorg.es
<ul><li><strong>Target version</strong> changed from <i>1.5.2</i> to <i>1.5.3</i></li></ul> CUNY Academic Commons - Bug #2772: Fellowship website layout bughttps://redmine.gc.cuny.edu/issues/2772?journal_id=130242013-09-15T21:03:30Zerin glasserin.glass@gmail.com
<ul></ul><p>Thanks so much to both of you! Site looks great in Firefox! Will check other browsers at another computer.</p> CUNY Academic Commons - Bug #2772: Fellowship website layout bughttps://redmine.gc.cuny.edu/issues/2772?journal_id=130312013-09-17T13:43:58ZDominic Gigliohumanshell@gmail.com
<ul><li><strong>Status</strong> changed from <i>Assigned</i> to <i>Resolved</i></li></ul><p>Glad we got it fixed and working Erin.</p>
<p>I've checked in Safari, Firefox and Chrome on my Mac and it looks good in all of them. If you find an error in another browser please open a new issue for that problem. I'm closing this one since it appears to be working fine.</p> CUNY Academic Commons - Bug #2772: Fellowship website layout bughttps://redmine.gc.cuny.edu/issues/2772?journal_id=130382013-09-18T15:35:02ZMatt Goldmattgold@gmail.com
<ul></ul><p>Fantastic - many thanks for your work on this, Dom.</p>