Problem/Motivation

Drupal Core now has a capability to generate a theme from another theme
https://www.drupal.org/docs/core-modules-and-themes/core-themes/starterk...

Steps to reproduce

Proposed resolution

  1. Make Basic a starterkit theme
  2. Remove all custom drush duplicate stuff
  3. Add documentation to README on how to use

Remaining tasks

User interface changes

API changes

Data model changes

Issue fork basic-3337052

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

joelpittet created an issue. See original summary.

occupant’s picture

Title: Convert Basic to use startkit theme » Convert Basic to use starterkit theme
Assigned: Unassigned » occupant

joelpittet’s picture

Status: Active » Needs review
Issue tags: +ContributionWeekend2023

We are going to remove the field.html.twig to the default stable. Which removes this code and BEM class syntax

# Create classes array #}
{% set classes = [] %}

{# BEM inspired class syntax: https://en.bem.info/
   Enable this code if you would like field classes like "article__tags", where article is the content type and field_tags is the field name.
#}
{% set classes = classes|merge([
  bundle ~ '__' ~ field_name|replace({'field_' : ''})|clean_class
]) %}
joelpittet’s picture

When generating the starterkit it copied over final class StarterKit by accident it seems because that would fail as it's the same namespace.

  • joelpittet committed b5493512 on 3.x authored by occupant
    Issue #3337052 by occupant, joelpittet: Convert Basic to use starterkit...
joelpittet’s picture

Status: Needs review » Fixed

Fixed, thanks for your help @occupant!

occupant’s picture

Re #5, the command I used was php core/scripts/drupal generate-theme basic --path themes/custom as per the docs.

joelpittet’s picture

Parent issue: » #3337115: Basic 3.0 Roadmap

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.