Problem/Motivation

Initially handled in #3517033: Add a style utility API.

As the goal is to split the issue into smaller with reduced scope, I create this issue.

Issue fork drupal-3631472

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:

Comments

grimreaper created an issue. See original summary.

grimreaper’s picture

Assigned: grimreaper » Unassigned
Status: Active » Needs review
smustgrave’s picture

Status: Needs review » Reviewed & tested by the community
Issue tags: +Needs Review Queue Initiative

Seems straight forward. Agree validate is not altering so makes sense

quietone’s picture

Title: Docs: Typo in IconPackManager » Fix description of the definition in IconPackManager::validateDefinition
Component: theme system » documentation
Category: Task » Bug report

This is a bug because the documentation is "Incorrect or misleading". And docs fixes should be in the documentation component.

  • quietone committed 5f407d2c on 11.4.x
    fix: #3631472 Fix description of the definition in IconPackManager::...

  • quietone committed 1c40907e on 11.5.x
    fix: #3631472 Fix description of the definition in IconPackManager::...

  • quietone committed ed175db1 on 11.x
    fix: #3631472 Fix description of the definition in IconPackManager::...

  • quietone committed 08f59d29 on 12.0.x
    fix: #3631472 Fix description of the definition in IconPackManager::...

  • quietone committed 9f359f93 on main
    fix: #3631472 Fix description of the definition in IconPackManager::...
quietone’s picture

Status: Reviewed & tested by the community » Fixed

@grimreaper, thanks for the fix, and keeping the other issue in scope.

Now that this issue is closed, review the contribution record.

As a contributor, attribute any organization that helped you, or if you volunteered your own time.

Maintainers, credit people who helped resolve this issue.

quietone’s picture

Version: main » 11.4.x-dev
grimreaper’s picture

Thanks @quietone for the proper category and the merge.