Code tour, 2/22-3/1

[denise]

A slow week this week with 7 bugs marked FIXED, but still a nice mix of frontend, backend, and user suggestions. This week, we welcome noodles, with his very first committed patch for us. (First of many, we hope!)

Come with us on a tour...



Bug 2140: Set Manage Filters to Subscriptions in the 'Organize' Menu
Category: Usability, dw_suggestions
Patch by: hotlevel4
Description: Suggestions discussion says that people are more likely to want to edit their subscription filters than their access filters, so this changes the landing target of the "Manage Filters" link in the site menu to subscription filters (with a link to editing access filters, so it won't be completely hidden if you're the other way around).


Bug 2216: add header comments on all files forked from LJ
Category: Backend
Patch by: denise
Description: Sometimes you need some calm soothing cut and paste. A lot of calm soothing cut and paste. This bug -- done over several months -- adds "this file originally written by LJ"-type headers to all of the files that we originally forked from LJ, in order to give them proper credit.


Bug 2376: Make icons link in sitescheme
Category: Usability, dw_suggestions
Patch by: denise
Description: Another Suggestions Special; this changes the site layouts so that a user's icon (in comment threads) links to their icons page. (It already did in many layouts, but not in the site layout.)


Bug 2405: remove support/see_overrides.bml.text
Category: Cleanup
Patch by: denise
Description: I take entirely too much glee in getting rid of old files that don't apply to us anymore. This removes some of the text used for a support-related page; it allowed support volunteers to view a user's S1 overrides for troubleshooting, and we don't support (or have!) S1.


Bug 2408: add to diversity statement
Category: Site documentation, dw_suggestions
Patch by: denise
Description: Someone suggested through the suggestions process that we add "neurotype" to the Diversity Statement; since putting something invovling inclusion and social-justice matters up for public discussion is potentially hurtful and there's zero reason for us not to do it, we didn't bother with the discussion part. (And if you spot a you-shaped hole in the Diversity Statement -- an identity that applies to you that we haven't included -- please always feel free to drop me an email and let me know about it.)


Bug 2409: Clean up remaining BML::ml usage in LJ::Poll
Category: Backend cleanup
Patch by: noodles
Description: This is deep voodoo. All right, all right, it's really a way to make sure that people can view and manage polls in ways other than through the website, like in downloadable clients, but really, isn't "deep voodoo" just so much more fun to say?


Bug 2410: Rephrase site style/journal style checkbox in customize pages
Category: Usability
Patch by: denise
Description: So, on the Customize Journal Style pages, there's that box in the options that says "Show entry pages in my journal style rather than the app style". That's the box that governs whether or not your "single entry" pages shows in the site layout or in your journal layout -- usually called "custom comment pages" -- and absolutely nobody can ever find it. So, with this patch, it is being renamed to "Show entry pages in my journal style rather than the site layout", which should help.

Comments

[zvi]

it allowed support volunteers to view a user's S1 overrides for troubleshooting, and we don't support (or have!) S2.

Er, that should be s1.

[denise]

*waves hand* These are not the droids you're looking for. Move along.

[feathertail]

Yay for neurodiversity. ^.^ Many thanks.

[cesy]

So much joy for 2376.

[liv]

OMG you fixed the "app style" thing! I know it's a tiny tiny thing but I <3 you for finally getting rid of that incomprehensible copy. I just wrote a FAQ explaining what on earth that means, and I have never been happier for my documenting effort to be made redundant.