Drupal/Core/Utility/Color does not have any dependency on Drupal, only on Drupal\Component, so let's move it there.

CommentFileSizeAuthor
#1 2324113-color-1.patch3.14 KBpenyaskito

Comments

penyaskito’s picture

penyaskito
he/him
Primary language Spanish
Location 🧑🏽‍🌾 Seville 💃 Andalusia, UTC+2 🇪🇺
commented
Status: Active » Needs review
StatusFileSize
new 2324113-color-1.patch3.14 KB
dawehner’s picture

dawehner
Primary language German
commented
Status: Needs review » Reviewed & tested by the community

Nice catch! The only dependency is to unicode, which is already a component!

alexpott’s picture

alexpott
he/they
Primary language English
Location 🇪🇺🌍
commented
Status: Reviewed & tested by the community » Fixed

Committed a7364b3 and pushed to 8.0.x. Thanks!

  • alexpott committed a7364b3 on 8.0.x 
    Issue #2324113 by penyaskito: Move Drupal/Core/Utility/Color to Drupal/...
dawehner’s picture

dawehner
Primary language German
commented

It seems to be that we never had a change record for the color methods. I guess we want to add one then.

penyaskito’s picture

penyaskito
he/him
Primary language Spanish
Location 🧑🏽‍🌾 Seville 💃 Andalusia, UTC+2 🇪🇺
commented

As far as I see (git show 5789200), these were code additions and not moving stuff around. Do we need change records for helper additions too?

Status: Fixed » Closed (fixed)

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