fu: Close-up of Fu, bringing a scoop of water to her mouth (Default)
fu ([personal profile] fu) wrote in [site community profile] dw_beta2012-03-24 01:20 pm

New JS on Journals: Last Call

We're planning on taking the "new JavaScript on journals" changes out of beta soon and making the changes the default for the entire site, so we've done a push today that adds the last few (reported) missing features, and fixes the last few (reported) really annoying behaviors.

There's always going to be one thing more though, so please make sure that you turn on beta for the new JS on journals, and tell us how that new JS works for you -- or doesn't work, if that's the case.

I'm interested in hearing about everything that's bothering you, no matter how tiny it seems! The sooner we know, the sooner we can fix it. (Just this code push, we fixed two things that seemed tiny: one was the hover menu fading way too quickly, another was how the attempt to change your poll vote would bring to you to a separate page instead of changing your vote on the page you were on. They were both things we missed at first, but once they were brought to our attention, we couldn't stop seeing them. And now they're fixed!)

So poke around, and let us know what you find. If you're looking for something to focus on, here's a short list of things we recently added or fixed:

  • added same-page comment tracking

  • fixed same-page changing of poll votes

  • added displaying errors when comment expansion fails to work

  • fixed hover menu fading away even when you didn't move your mouse

  • fixed the .cuttag-open CSS class so it applies to the contents of the opened cut tag (the way the old JS did), for people who want to customize this in their styles

  • made the bottom arrow jump you to the top of the cut tag, without affecting page forward/back



Assuming there are no major issues, we'll turn on the "new JS on journals" beta flag for everyone with our next code push for one last round of testing with the whole site, not just the people who've turned on the beta version. We'll keep it as a beta option at that point, so if anyone has serious problems, they can disable the beta and go back to the old JS while we fix the problem. After that, we'll take this code out of beta and make it the standard option for everyone, removing the old (existing) JavaScript entirely. So, if something's wrong, speak up now!
thenormalsquint: (Default)

[personal profile] thenormalsquint 2012-03-25 07:53 am (UTC)(link)
Directed here by Denise. I cleared my cache and cookies, rebooted both my computer and my browser and the problem with Expand All hanging and freezing remains. I'm on Firefox 7.0.1 with Adblock, Adblock Element Hider, and Ghostery (which I tried whitelisting DW on thinking it would help, but nope.)
denise: Image: Me, facing away from camera, on top of the Castel Sant'Angelo in Rome (Default)

[staff profile] denise 2012-03-25 05:23 pm (UTC)(link)
[personal profile] eyesofstrength, below, was having a similar problem and narrowed it down to three extensions (tab counter, tab mix plus and tabs menu) -- do you have any of those installed? If not, can you try the exclusion method I described in this comment and see if you can narrow down which extensions are causing the problem?
pocketmouse: pocketmouse default icon: abstract blue (Default)

[personal profile] pocketmouse 2012-03-25 06:02 pm (UTC)(link)
I was getting a jquery error (which I screencapped, if you want it). Firefox 11, Mac 10.5.8. I have Tab Mix Plus, so I disabled that and restarted, but then it threw errors at me from other addons, specifying the addons this time, specifically for DoNotTrackPlus and HTTPS-Everywhere. With those two disabled but Tab Mix Plus on, it works, if a bit slow. If I turn one of them on, it throws a jquery error at me, not an add-on error.

ETA: And then it did it again, this time telling me there was an error in ghostery. Basically, it's not happy with any of my security addons.
Edited 2012-03-25 18:08 (UTC)
denise: Image: Me, facing away from camera, on top of the Castel Sant'Angelo in Rome (Default)

[staff profile] denise 2012-03-25 06:58 pm (UTC)(link)
Huh. Yes please, seeing errors is always helpful!
pocketmouse: pocketmouse default icon: abstract blue (Default)

[personal profile] pocketmouse 2012-03-25 07:06 pm (UTC)(link)
JQuery error (it was the same one every time)

Various addon errors.

None of the listed errors are for Tab Mix Plus, so I really don't think it's related.
pocketmouse: pocketmouse default icon: abstract blue (Default)

[personal profile] pocketmouse 2012-03-27 11:56 am (UTC)(link)
Nope, sorry. I could try and find some custom comment pages with enough entries to expand, but it wouldn't be til this evening.
funkybrewsta: (Default)

[personal profile] funkybrewsta 2012-03-26 04:48 am (UTC)(link)
I have neither of those add-ons, though I do have IE Tab and ColorfulTabs which I turned off only to get MORE errors from Chrome (I don't know why when I am using Firefox) and Ghostery and ooh boy. Like Pocketmouse said, it seems unhappy with anything.
thenormalsquint: (Default)

[personal profile] thenormalsquint 2012-03-27 08:58 am (UTC)(link)
Just site skin comment pages since I haven't run into any custom comment pages yet to test. As for your other comment, I get unresponsive script errors and a couple others. If you'll give me a few minutes, I can get some screencaps going.
thenormalsquint: (Default)

[personal profile] thenormalsquint 2012-03-27 09:13 am (UTC)(link)
Okay, this one is using Expand All on this Entry.

These are the ones I got using the entry on this community titled New Create Entries beta, round two:

One
Two
Three

Does that help any? I couldn't replicate the error that said something about chrome unfortunately.