Published: August 16, 2024
My LocalGov Drupal contributions for week-ending August 16th, 2024
Here's what I've been working on for my LocalGov Drupal contributions this week. Thanks to Big Blue Door for sponsoring the time to work on these.
Need LocalGov Drupal services? Give Big Blue Door a shout.
I only got around to working on LocalGov Drupal for one day this week, but we have some really nice work done in this. I'm especially happy with item 4 below which fixes some accessibility issues.
Meetings and Meet-ups
- Tech Group Drop-in: I was the only core maintainer at this week's meeting, which was kinda scary and kinda cool all at the same time. We discussed publications, permissions, deployments, upgrades and ... why do we have so many updates!
Code Contributions
- I have an MR for the Preview Link module so we can copy the preview link to our clipboard. A new release of Preview Link module has meant that my patch no longer applies, but the merge conflict is just one line, so it should only take a second to fix. Right? Wrong - I've no idea what happened but it was one of those days when trying to run the core linting tests on my MR caused my DDEV to crash and crash and crash. Well, I eventually got DDEV working again, but couldn' get the core JS tests to run against the patch. You win some, you lose some.
- We have a "sticky header" PR open for LocalGov Base for a while now. It's close to being ready for merging, except we have an accessibility issue where links might be behind the sticky header so you can't see what you are focussed on. The bigger issue is I can't replicate it in order to get a fix for it. I'd be happy if you could do so and comment on the issue.
- Rupert from Chicken created a pretty nifty issue in LocalGov Base to add a new variable that reverse CMSs could use to render breadcrumbs. The specific use-case is for the ModernGov templates, but anyone else could also use it - such as Netcall. And true to form with Rupert, it wasn't just an issue he created but also a pull request to go with it.
- Maria reported a humdinger of an issue concerning accessibility and focus-order for items in the header/services menu. I created a PR to fix them, let's hope we get this merged soon, better accessibility is better for everyone.