Published: October 3, 2024
My LocalGov Drupal contributions for week-ending October 4th, 2024
This week, I spent my time mostly looking at issues tagged with "Editor Experience".
Need LocalGov Drupal services? Give Big Blue Door a shout.
Meetings and Meet-ups
- Merge Tuesday: We still don't have LocalGov Base 1.7.0 released, but are very close once we get the adjacent PRs for LocalGov profile and Microsites profile all merged at the same time. Hopefully we'll get there by the end of the week.
- Accessibility Meetup: Maria from Agile Collective gave a really good presentation about the use of colour on our websites. And we recorded it, so do go watch it back.
- Tech Group Drop-in: I hosted the meeting again this week - I'm getting used to that now - and we talked about facets, carousels, and default themes for microsites.
Code Contributions
- We've been going around and around with an issue for moving the "Files" tab from the content tabs list to the media tabs list. We decided at the Product Group meeting last week to add an update hook to move it for existing sites as well as new installs, and create release notes to let people know why. So ... that's done now, hopefully we'll get it merged at Merge Tuesday tomorrow.
- Another kind of lingering issue was adding masquerade module vs msqrole module to our default installation. At the Product Group meeting last week we decided to go with masquerade module since it now has security team coverage and is the preferred choice from the Drupal Startshot initiative. With that being the case, we now have a PR for this.
- I updated last week's PR to enable the LocalGov Base Helper module for LocalGov Microsites Group, so it now does it via an update hook instead of in the
info.yml
file. - I enjoyed spending some time this week researching solutions for a "What other pages on my site link to this page?" feature that we want to add to LocalGov Drupal. What wasn't quite so enjoyable was my findings - a module to do what we want, in the way that we want it, does not exist.
- Alistair Wearing has written a nifty PR for us to add the icons for accordions via Twig templates instead of JS so it is much easier to override and maintain if you want to. Once we get that merged, I've a few follow-up issues I'd like to consider as well - such as placing the title of the accordion inside the button and placing the icon for the accordion outside the button.
- Our 1.7.0 release of LocalGov Base is going to have a helper module along side it, so we can have update hooks in themes, basically. But this is needing a lot of coordination between the default profile the default theme the microsites profile and the microsites theme. After a lot of reviewing of a PR on microsites, it looks like we're just about there now.
- Paul McCrodden has been helping us out with some PRs, working through some of the "Good first issue" items to get himself up to speed with how we work. It's been great to mentor him and help him find issues to work on, and even approve some of his PRs. Thanks Paul.
- And that then got me thinking about a larger clean-up issue for us - use logical properties for all our CSS. This will mean a smaller codebase to maintain, but also an easier codebase to maintain. No more
[dir="rtl"]
styles. - We are working hard on "Editor Experience". One issue was trying to make it easier to link to documents from your media library. I've created a PR for this now, so you can add simple links to documents. If you want a more complex display of the document (including metadata, etc) you can still do that via the standard media embed.