Feedback from @justafish on #3079738-75: Add Claro administration theme to core:

Some files are prefixed with claro. and others aren't, let's just go with 1 convention (my preference would be without the claro. prefix)

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

lauriii created an issue. See original summary.

ckrina’s picture

lauriii’s picture

Status: Active » Needs work

We will also have to update the new filenames to claro.libraries.yml and claro.info.yml 😇

ckrina’s picture

Status: Needs work » Needs review
FileSize
2.88 KB

Ooops, too fast 😂

  • lauriii committed dce40c2 on 8.x-2.x
    Issue #3087187 by ckrina, justafish: Remove claro. prefix from...

lauriii credited justafish.

lauriii’s picture

Status: Needs review » Fixed

Thank you!

  • lauriii committed 5625a33 on 8.x-1.x
    Issue #3087187 by ckrina, justafish: Remove claro. prefix from...

Status: Fixed » Closed (fixed)

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