Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.Problem/Motivation:
- We have the body styling established in the POC repo (https://github.com/lullabot/olivero-poc). This work is now ready to be moved into the Drupal theming process.
Proposed resolution:
Implement styling for the following body elements:
- Paragraphs
- Headings
- List (Unordered/Ordered)
- Table
- Inline links
What's missing:
To get the body area themed during the FL Drupal Camp sprint, we're going to leave out the media embed theming. This work will be addressed in a separate issue.
| Comment | File | Size | Author |
|---|---|---|---|
| #5 | 3115194-theme-body-field.patch | 1.06 KB | brianperry |
Comments
Comment #2
proeungComment #3
proeungComment #4
brianperryGoing to start working on this as part of the MidCamp sprint.
Comment #5
brianperryWith the attached patch you should be able to create a basic page with all of the content markup from the POC below the hero and it will be styled correctly. This covers the body elements listed in the description, along with other elements. I also made a minor adjustment to the page template to make the layout match the POC more closely. In a default install the breadcrumb block will disrupt the layout a bit - if you want everything to align, disable this block.
I took the approach of adjusting the template to match our existing css rather than the inverse. Seemed safer given that we're still integrating some changes from the POC. Not sure if that what we want long term - I'll bring that up during the Monday meeting.
Comment #6
mherchelThis looks great! Committed!