\Drupal calls should be avoided in classes, use dependency injection instead

Please review patch.

rakesh.regar → made their first commit to this issue’s fork.

Raised PR.

@chaitanyadessai: How is \Drupal::service('extension.list.theme') getting injected? I don't see where that $this->themeHandler is defined.

@rakesh.regar: How does your merge request differ from the patch? Have you contributed anything to this issue?

Please review updated patch.

The code looks like this:

    // @todo Searching the path is not what we want - this is how it was done
    // in D7, but that's not how assets should be handled in D8.
    $mailstyle = \Drupal::service('extension.list.theme')->getPath($theme) . '/mail.css';

I don't think we should be adding all the code to do the injection when this code is likely to go away entirely as a result of the @todo.

I'm postponing this issue pending the solution to the @todo in 📌 Add support for mail.css Active