title: Should modules that use HttpClient declare the HttpClientModule in imports?
body: <p><a href="https://github.com/angular/material.angular.io/blob/master/src/app/shared/svg-viewer/svg-viewer.ts" rel="nofollow noreferrer">The SVGViewer component</a> in the Angular Material documentation app declares its module like this:</p>
  
  <pre><code>@NgModule({
    exports: [SvgViewer],
    declarations: [SvgViewer],
  })
  
  </code></pre>
  
  <p>Is it "More" correct for this module to declare:</p>
  
  <pre><code>  imports: [HttpClientModule]
  </code></pre>
  
  <p>Since it uses <code>HttpClient</code> ?  IIUC in general modules should declare their dependencies.</p>