Code Tour: 2018-09-01 to 2019-04-29

[momijizukamori]

So it's been a while!


Issue 1691: Banning an OpenID user exposes their ext_?? username. (pull request)
Category: Reducing Confusion
Patch by: Darael
Description: When you went to ban an OpenID user, the confirmation pop-up would use the 'ext_#####' designation they have in the backend system, instead of a human-friendly display name. This now uses the display name instead!

Issue 1913: Replace S2 layer editor with something slightly more modern (pull request)
Category: Styles, Getting the Coding Up To Plum
Patch by: rshatch
Description: We had a text editor for S2 layers that was... basically unchanged from what we inherited from LJ (and at that point, was unchanged since its creation, probably). As such, it mostly worked, but was pretty basic, and none of the extra bells and whistled functioned any more. This patch replaces that text editor with a new one with shiny features such as: syntax highlighting! Hotkeys! Auto-complete! Functioning links to documentation!

Issue 2230: TT/Foundation the logout page (pull request)
Category: TT Conversion
Patch by: zorkian
Description: One more step in our slow migration away from (old, badly documented) BML to (new, cleaner) Template Toolkit pages.

Issue 2249: supportdelete priv no longer exists, if it ever existed in GPL'd livejournal code. (pull request)
Category: Code Machete
Patch by: Darael
Description: There was user privilege called 'supportdelete'. It did not actually do anything, and possibly did not ever do anything. This removes it from the list of privs admins can assign, to reduce confusion.

Issue 2265: Basic API + sample routes (pull request)
Category: API, Getting the Coding Up To Plum
Patch by: rshatch, allen
Description: Among the things we inherited from LJ was the XMLRPC API interface. This was ancient, did not do a whole lot, wasn't maintained, and was largely undocumented. In short: bad. Also XMLRPC is a pain to deal with, and is increasingly unused for that reason. We now have all the backend code in place for a new, better-documented, easier-to-use JSON interface, as well as API keys for it and a few basic endpoints as examples. Once we're sure we haven't missed any major issues with this, we can start adding more complicated endpoints, but it's best to make sure the foundation is rock-solid first. This is actually a TON of backend work to do all the setup, and means that the user-facing bits (endpoints) will be a lot faster to create.

Issue 2278: Global backend changes to allow converting /customize to TT (pull request)
Category: TT Conversion
Patch by: rshatch
Description: While we had fancy TT elements for most form components, we were missing one for the color picker used in /customize. Also, it turned out those pages needed a few tweaks on the backend, like the ability to set form elements to nothing, and to be able to next template blocks inside themselves.

Issue 2283: Customize/advanced to tt (pull request)
Category: TT Conversion
Patch by: rshatch
Description: Beginning the long messy path to total TT dominance of /customize by starting with the /advanced pages (which, ironically, were less complicated code to change)

Issue 2356: remove Yahoo Messenger from user profiles (pull request)
Issue 2376: more "Other Sites" upkeep: remove Delicious (pull request)
Category: Profile, Dead Site Graveyard
Patch by: rshatch
Description: Some services have vanished since we last updated our profile pages, and it doesn't make sense to keep listing the ones that don't work any more. These fixes remove Delicious and Yahoo Messanger.

Issue 2358: add discord to profile #2358 (pull request)
Category: Profile, Getting the Coding Up To Plum
Patch by: srukle
Description: ...And some new sites and services have become popular in their place! Now you can put your Discord username in your profile and have it show up nicely under 'Other Sites'.

Issue 2377: whitelist embeds from streamable.com
Issue 2366: add pastebin.com to embed whitelist
Issue 2397: add gimletmedia.com to embed whitelist
Issue 2400: add random.org to embeds whitelist
Issue 2403: whitelist embed new spotify (pull request)
Issue 2402: whitelist embed from strava
Category: Embedding
Patch by: rshatch
Description: People like to show content from other sites in their journals! Now the list of sites you can put stuff from into Dreamwidth pages is longer. And Spotify works again!

Issue 2368: Attempt to make Travis run compiler (pull request)
Category: Getting the Coding Up To Plum
Patch by: zorkian
Description: Adds an integration to TravisCI, so that when people submit pull requests we can run very basic tests to make sure it won't make absolutely everything fall over if we use that code.

Issue 2372: Convert /latest to TT and spruce it up a little. (pull request)
Category: TT Conversion
Patch by: rshatch
Description: So in 2012, I told Mark that if he added Markdown support, I would clean up /latest for him. He added Markdown support that day, and I.... have finally fulfilled my half of the bargain. /latest is now using TT, and has been polished up a little.

Issue 2373: Move stc include to the controller so it's loaded for cached version.… (pull request)
Category: TT Conversion
Patch by: rshatch
Description: It turned out that including CSS in the template of /latest didn't work when it was cached (and caching is good, it makes things faster!), and it had to be in the controller for that page instead. This makes that move.

Issue 2378: properly anchor header navlinks in Tectonic (pull request)
Category: Styles
Patch by: rshatch
Description: If you used header navlinks, and made your header really tall in Tectonic, the navlinks would end up kind of.... floating in the middle. Whoops. Now they stay where they're supposed to be.

Issue 2381: remove fastclick.js (pull request)
Category: Code Machete, Getting the Coding Up To Plum
Patch by: rshatch
Description: Fastclick.js was originally a hacky workaround for an issue with very early versions of Android - on pages not flagged a special way for mobile use, it would delay sending the message that an item had been clicked on for half a second on touchscreens (which is long enough to be noticable). Fastclick.js got around this by making a pretend mouse click in the same spot as the touch click. Which.... worked, mostly. Except Android got its act together, and this library was abandoned, and it turned out it was causing all sorts of bizarre behavior on mobile devices. We had it because it came bundled with Foundation, but nothing else relies on it, so away it went!

Issue 2386: Update year in site footer
Category: Getting the Coding Up To Plum
Patch by: rahaeli
Description: It's 2019, yeaaaaah! Except our site still said it was 2018 in the copyright notice in the footer. This is now fixed (and was made live on production already)

Issue 2390: Librarian's Dream style doesn't use header image properly (pull request)
Category: Styles
Patch by: rshatch
Description: Librarian's Dream has the option to use a header image, but... turned out it was getting covered up by blocks of header color because of the way it was written. Now you can add a header image without having to do a bunch of faffing around with CSS to make it actually show up.

Issue 2391: Don't strip meta tag when 'use layout stylesheet' is unchecked (pull request)
Category: Styles, Getting the Coding Up To Plum
Patch by: rshatch
Description: It turns out we were removing the meta tags that make pages display nicely on small screens when people used custom CSS instead of the layout stylesheets. This was under the assumption that custom CSS layouts were probably not optimized to behave correctly on small screens, which may have been true at one point, but it's now 2019, and people are writing custom layouts with small screens in mind. And there was no way to add the meta tag back in without either including all the layout stylesheets (extra stuff to load) or manually editing S2 layers (finicky and basically a huge pain). Now all layouts include the meta viewport tags, unless you have explicity disabled it (Account Settings > Display > Mobile View > check 'Turn off mobile view for this device')

Issue 2396: Tagnav highlighting not always working (pull request)
Category: Reducing Confusion
Patch by: alierak
Description: We have a feature where you can navigate entries within a tag. It's supposed to keep the tag you're browsing by highlighted, for easier navigation, but this wasn't actually working when the tag had more than one space, or special characters. Now it does!

Issue 2399: Add classes to allow easy targetting of nav module list elements (pull request)
Category: Styles
Patch by: rshatch
Description: One of the most common questions asked on styles_system is 'how do I hide [x] nav menu item?'. Before, there wasn't a particularly clean way to do it that wouldn't either leave weird extra spaces, or be prone to breaking. Now each of the menu items has their own CSS class so they can be targetted and hidden easily.

Issue 2401: typo in faq #175
Category: Docs
Patch by: kaberett
Description: Typos happen to the best of us, but now FAQ #175 will have one fewer!

Issue 2412: checkconfig.pl: don't re-invent version comparison (pull request)
Category: Reducing Confusion
Patch by: nfagerlund
Description: Our script that checks that required dependencies are installed and up to date had some homegrown logic in it that was causing it to report that version 1.5 of something was not up to date with the requirement of version 1.45 (because 5 is less than 45), despite it being a higher version (because in the Perl world, 1.5 is the same as 1.50). We probably needed the homegrown version when the code was originally written fourteen years ago, but Perl now has a nice way to get version numbers for comparison that actually correctly reports 1.5 as being newer than 1.45.


28 total issues resolved
Contributors: Darael, allen, alierak, kaberett, nfagerlund, rahaeli, rshatch, srukle, wahiaronkwas, zorkian - welcome back to a few old faces, and a special shout-out to nfagerlund for their first contribution!

Comments

API and CI!!!!

[brainwane]

Thank y'all as always! And in particular I am excited to read:

We now have all the backend code in place for a new, better-documented, easier-to-use JSON interface, as well as API keys for it and a few basic endpoints as examples.

Adds an integration to TravisCI, so that when people submit pull requests we can run very basic tests to make sure it won't make absolutely everything fall over if we use that code.

AW YEAH! Rock!

Re: API and CI!!!!

[momijizukamori]

I am SO HYPE about the new API, that's been my baby for a while now. Also the new S2 editor even though I know the number of people who care about that is much smaller - I've been talking about doing that since ~2013.

Right now the TravisCI checks are VERY basic, as in, installs dependencies and runs our config check script and nothing else, but in the future it can maybe be shinier!

[20_00]

Issue 2265!

Sorry, I did not understand.
This means that Semagic will stop working ??! If so, very bad (

[momijizukamori]

No, you're fine for now! We aren't removing the old API that Semagic uses - while we might in the future, we'll be sure to give plenty of notice, and it'll definitely only be after the new API is complete and usable for new apps. For now the two will live side by side.

[cesy]

I am excited about the API and the S2 layer editor and the TT-ification and general cleanup.

[momijizukamori]

ME TOO.

[azurelunatic]

Whee!

[roadrunnertwice]

sweeeeet

In retrospect I guess the title of that PR should have been "don't pre-invent version comparison," lmao

[momijizukamori]

Yeah, LJ did that a bunch of times. And then like, never properly documented the work for other people to build on, so it got left in the dust (see also: BML)

[mdlbear]

YAY! Looking forward to playing with the new API.

[momijizukamori]

It doesn't do a lot yet, but now that all the backend pieces are in place.... soon.

[arethinn]

Description: People like to show content from other sites in their journals! Now the list of sites you can put stuff from into Dreamwidth pages is longer. And Spotify works again!

Do I need to be using the standard (non-beta) Create Entries page to see such embed options?

[alyndra]

You just need to copy-paste the html for embedding into the entry space, it shouldn't matter which page you're using!

[musyc]

Issue 1913: Replace S2 layer editor with something slightly more modern

Ooooh? OOH. Can't wait to poke at this!

[momijizukamori]

I'm like, unreasonably proud of that one, maybe.

[jazzyjj]

Just read through this latest code tour, and am excited to try out the enhancements once they go live. I'm a bit confused though about what you mean by S2. Does this mean that the beta "Create Entry" page is now coming out of beta? I've been using that for awhile now because it is just slightly more accessible with my 2 screen readers than the original default editor. Not by much though. But anyways, thanks again for all the time and effort that has gone into Dreamwidth! Very much appreciated.

[denise]

S2 is the journal customization programming language that runs under the hood -- some people like to get really detailed in their customization by writing new styles rather than using the site styles and just tweaking them a bit. You probably will never have to worry about it!

[alexseanchai]

discord! yay!

nice usericons

[alexseanchai]

my profile now says my discord is AlexSeanchai%235912 and I def told the form AlexSeanchai#5912

...this probably goes in dw_maintenance?

[lxe]

I am feeling a little bit uneasy about removing old messenger credentials. While unusable for direct messaging, they might still be useful for reconnecting with old friends etc. I understand that it might sound like deadnaming, but - if anyone would like to remove their profile element, they would; and if they'd like to retain one, it shouldn't require extra effort.

On the third hand, I fully admit that I have no standing here; my published contacts are up to date (or, more accurately, were moments ago). Just nitpicking.

[denise]

We remove old/defunct sites because the list is really long already and people don't deal well with long lists of inputs -- every text box you add, it's one more text box you need to scan through to find the one you want. It's not about the visual clutter on the profile page that people see when they look at their profile, it's about the clutter on the management page when someone is trying to edit their profile!

[silveradept]

These look like very useful things to people who know what to do with them.

[momijizukamori]

They are, and hopefully the things people make with them end up making stuff better for everyone :)

[cpthorn]

Thank you! Everything works great so far. Hope the new Create Entries page will be released soon, it's still in Beta, but I've been using it on a permanent basis for a long time. The only thing I'm missing is the ability to quickly insert <cut text="..." and <img src="..." tags from the panel. Also the ability to search for mobile-friendly styles would be very useful.

[momijizukamori]

That's one of those so close items - we're definitely keeping it on the radar, for sure!

[azurelunatic]

The text on my profile page is now in exciting different sizes. I draw attention particularly to AO3 & Discord in the other services bit.

[momijizukamori]

Did the hotfixes for discord last night fix it? It all looks the same size to me.

[hitchhiker]

ooh, awesome! really looking forward to seeing where the api cleanup goes.

[momijizukamori]

Exciting and useful places, I hope!

[saga]

Hi Team! Thanks for the hard work and the additional icon slots IT'S NICE. I also like how viewport meta tag is also gonna be usable in custom.

I use the Tectonic layout for one of my accounts and after the code push, it's pushed everything up and the headers now float all over the navigation strip. I also use Firefox. Is there a setting I am missing? Thanks!

[momijizukamori]

This sounds like a bug! I will see about having a look later today.

[syntheid]

Oh man this is just full of good news. New api backend! TravisCI! S2 editor update!

[momijizukamori]

Many shinies!

[sylvaine]

Hooray, thank you everyone! \o/

[momijizukamori]

Yes!