Published: September 5, 2024
My LocalGov Drupal contributions for week-ending September 6th, 2024
One of those weeks where we got lots and lots of smaller issues cleared up, and a new module released, and a very quirky bug discovered.
Need LocalGov Drupal services? Give Big Blue Door a shout.
Meetings and Meet-ups
- Merge Tuesday: Lots of talk this week about supporting Drupal 11 and what we need to do for our automated tests to work for it, while also supporting Drupal 10.
- Accessibility Meetup: Michael from Big Blue Door gave a really interesting presentation on the tools he uses for checking the accessibility of their websites, notably Axe, Wave, and Lighthouse, as well as manual testing and keyboard navigation testing.
- Tech Group Drop-in: We had two big juicy topics today, nice to get down and dirty with tricky stuff - media downloads and empty regions + caching.
Code Contributions
- Christopher had a really nice PR created for me to test on Monday morning. The PR would allow our accordion patterns to have more than one item open at a time and also add a toggle to 'Show all/Hide all' accordions. Each of these features can be turned on/off for individual accordion containers. Very nice.
- Lee suggested that the LGD Dashboard I built a couple of weeks back should have a link for "Good first issue", so now it does.
- We've had an issue for a while now about removing the 'Files' tab from the list of tabs you see on the admin/content page. A lot of content editors are clicking on it expecting that's where their "Files" are, which it is, except what they are actually looking for is their "Media" items. We had a PR to remove this tab altogether, but that will break existing sites, and some users might want to visit it. So. we now have a PR to set the "Files" tab to be a sub-item of the "Media" tab. That makes sense I think.
- I had forgotten we had a module called "LocalGov Extra Layouts". Justine created it originally when she needed a 4-column layout, but we then added 4-column layout to the LocalGov Paragraphs module. The other layouts in the LocalGov Extra Layouts module are probably still useful to other councils - 33:66, 66:33, 2x2x2x2, etc. I did some work on these to close out an issue created by Andy saying the module had not been added to packagist (it is now, along with its own shiny composer.json file) - this was the issue that brought this module back to my eyes.
- Then I did some work to tidy up the code - fix the package name, tidy up the CSS, remove the redundant template, etc.
- LocalGov Extra Layouts had no release branch and no tests, so I created an issue to fix this and then a PR for it to use the shared workflows approach we have.
- What module would be complete without a README.md? Well, here's a PR for LocalGov Extra Layouts to now have one.
- Issue number 1 for the LocalGov Drupal Dashboard was to create a button that would copy the clone command for a repo to your clipboard. And now we have it.
- Whilst there, I did a little clean up of the table layout and the general cards layout.
- I had a good chat with Sarah from Blackburn with Darwen Borough Council and we think we've discovered a bug in the path alias settings for publications. I haven't got it fully uncovered yet, and I certainly don't have a fix for it yet, but I've some notes written up about it, and we'll get it solved from there.
- Following that call, I put together a video to demonstrate the bug. Hopefully that will help us to get a fix for it quickly.
- Zach posted a PR to link the SVGs on publication cover pages to their respective files. We got it reviewed and merged.