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.
We need to start converting the bartik page.tpl.php into a valid layout plugin. This will likely include the addition of a few new regions for areas like breadcrumbs that were hardcoded into a section of the layout.
Comment | File | Size | Author |
---|---|---|---|
#1 | 1929688-1.patch | 11.92 KB | EclipseGc |
Comments
Comment #1
EclipseGc CreditAttribution: EclipseGc commentedThis is a first pass at this that I mostly stole from sdboyer's excellent patch over on #1812720: Implement the new panels-ish controller [it's a good purple]. I figured I could try to push in smaller components of what he's doing that should be relatively non-controversial. This is a first pass, and some portions that shouldn't be in a layout plugin are still there. Anything that's not a full fledged region needs to be address and either removed, or replaced with a region (depending upon whether it sits in a portion of the template that a region couldn't just as easily fulfill. breadcrumbs is a good example of something that will likely need a custom region).
This is throwing a few errors still, but we need to address the functionality that's trying to provide in any event, so I didn't try to fix it yet.
Eclipse
Comment #3
lokapujyaIs this something that can still be done or was it dependent on layouts?
Comment #4
Jeff Burnz CreditAttribution: Jeff Burnz commentedGiven that #1787846: Themes should declare their layouts is closed I think we can assume this should close also.