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.
In order for Zen to register as a Drupal 8 theme, we must port the .info theme code from D7 to the appropriate YAML standards for D8.
Comment | File | Size | Author |
---|---|---|---|
#10 | zen-yaml-migration-2487856-10.patch | 22.94 KB | kmonty |
#5 | D8-yml-2487856-5.patch | 14.43 KB | jamessw |
#2 | D8-yml-2487856-2.patch | 14.43 KB | jamessw |
zen.png | 94.8 KB | marthinal | |
zeninfo.patch | 3.16 KB | marthinal |
Comments
Comment #1
JohnAlbinComment #2
jamessw CreditAttribution: jamessw commentedHere's a patch including the starter kit. At this point, you can enable Zen as the default theme, but it's not themed.
Comment #3
jamessw CreditAttribution: jamessw commented(Deleted-duplicate entry)
Comment #4
mkostrzewa CreditAttribution: mkostrzewa commentedShould .info be .info.yml?
This may need to be reworded to make it clear to the user that asset libraries are declared not JS and CSS files themselves.
Again, .info.yml
Should this be Drupal 8 core?
This may not be applicable anymore as we are using 'stylesheets-remove'?
This needs to be replaced with D8 specific info e.g .php.tpl files need to be twig.html and the php template syntax needs to be replaced with Twig's syntax.
This may no longer be applicable to D8. See https://www.drupal.org/node/2382645#comment-10380143
Should a version be added? https://www.drupal.org/theme-guide/8/assets
Link to the documentation might be good too.
Should there be a comment added regarding declaring jQuery as a dependency as it is no longer loaded by default on all pages?
As mentioned before, may not be applicable in D8 anymore.
Comment #5
jamessw CreditAttribution: jamessw commented@mkostrzewa thanks for your checks! I fixed the typos, but I'd appreciate if someone with more understanding worked on the documentation and the D8 changes. I'm wondering if we should make another issue for documentation, to be completed when work on Zen for Drupal 8 is closer to done, and we have a better idea of what needs to be documented?
Comment #6
mkostrzewa CreditAttribution: mkostrzewa commentedI assume this will be amended by the documentation person?
This needs to refactored for D8 please see https://www.drupal.org/node/2382645
As mentioned above will need to be refactored for D8
Comment #7
mkostrzewa CreditAttribution: mkostrzewa at PreviousNext commented@jamessw you're welcome. I added some comments to your latest patch. @JohnAlbin, should we add a ticket for documentation?
Comment #8
mkostrzewa CreditAttribution: mkostrzewa at PreviousNext commentedComment #9
kmontyPatch forthcoming.
Comment #10
kmontyI have Zen core working on my local, but have not updating the STARTERKIT YAML files yet.
Note: I removed panels (it's not ready for D8) and registry rebuild (in core now, yay!) from the settings. Will open separate tickets about that.
Comment #11
mkostrzewa CreditAttribution: mkostrzewa at PreviousNext commentedHi @kmonty, is that patch ready for review or are you adding another patch?
Comment #13
JohnAlbin