Published: August 9, 2024
My Drupal Core Contributions for week-ending August 9th, 2024
Here's what I've been working on for my Drupal contributions this week. Thanks to Code Enigma for sponsoring the time to work on these.
![Code Enigma Logo](/static/a879d973ce6df357af4e842bf27ef692/47930/code-enigma-logo.jpg)
Last week I was working on a merge request to add the new navigation module to the umami profile. If this is to be merged, then we won't need the toolbar module any more. So I updated that merge request to not install the toolbar module and to also remove toolbar-related permissions from authors and editors.
But after that, things got weird. Well, not weird, but more difficult. We have a message that shows on the toolbar to say "This site is intended for demonstration purposes.", but since we don't have a toolbar any more, well ... what happens to the message? Also, we have tests to make sure the message is in the toolbar, so those tests are now failing.
The options, as I see it, are:
- Remove the test for the text “This site is for demonstration …” and remove the text “This site is for demonstration …”.
- Place the text “This site is for demonstration …” somewhere else and update the test.
Update:
I think I have this is a good position now.
- Remove
toolbar
from modules to enable in demo_umami.info.yml - Add
navigation
to modules to enable in demo_umami.info.yml - Move "
This site is intended for demonstration purposes
" message from toobar to page_top - Rename
toolbar-warning
library and classes to 'demo-profile-warning' and classes - Remove
access toolbar
from permissions intestDemonstrationWarningMessage
test - Update permissions for author and editor
- Update the CSS so there's less of it, it's easier to read and we have the same CSS for small screens and large screens
I'm really looking forward to getting this merged to core, a much larger issue than I had originally intended it to be.
Thanks to <a href='https://www.codeenigma.com/'>Code Enigma</a> for sponsoring my time to work on this.