2009-10-07

denise: Image: Me, facing away from camera, on top of the Castel Sant'Angelo in Rome (Default)
[staff profile] denise2009-10-07 06:25 am
Entry tags:

Assigned-bugs hackathon!

I checked tonight, and we've got 571 open bugs. That's a lot! 146 of them are assigned with no patch, and if we could make some inroads on that number, we'd be able to knock down the total open number a hell of a lot.

So, here's my hackathon challenge! If you've got assigned bugs, let's do whatever we can this week to get that number down. I'll start; I have 8 bugs assigned to me without a patch, and while some of them are waiting on other things to get resolved first, some of them are things I've just been putting off because I don't know exactly how to do them or was running into problems.

I've already yelled for help on what I'm stuck on, so I'll turn to y'all: what can I help you with on stuff you're stuck on? And if not me, who else can help you? A bunch of the existing assigned stuff is really complex, so let's all work together and see how much we can improve things.

Ideally, I'd like to get that "open" number down below 500 by Sunday night, and double ideally, the "all unassigned" number (currently at 386) down under 300.

*salutes you all*
jd: (wax lips)
[personal profile] jd2009-10-07 10:28 pm
Entry tags:

javascript, the inbox, livejournal.js, and DOMContentLoaded

While I was looking at livejournal.js, I noticed that it uses the DOMContentLoaded event to start initializing things going before all the images and video content finish loading, instead of waiting for the window.load event (which occurs after everything finishes loading, including large files). But esn_inbox.js doesn't do this; instead, it waits until window.load to enable the controls, pagination, etc. This is a problem because if you have notifications that have a lot of large filesize content, like videos or large images, or if an image is hosted on a really slow server, you have to wait until it downloads (or times out) in order to be able to do anything with the inbox.

Is it possible to make esn_inbox.js use DOMContentLoaded, and fall back to window.load for browsers that don't support it (which, as I understand it, is anything not Mozilla-based)? I tried copying code out of livejournal.js into appropriate spots in esn_inbox.js, but I'm not entirely sure that this will work (I have no way to test any of this). Thoughts on this code? ) I'm asking here because I'm not sure if what I'm trying to do is even possible (or if it is, if it will work the way I think it will), and because I don't have a dev env set up. If this is fine, I can migrate to zilla, but I'm not sure how to generate a patch.