Remove redundant import in accordion component

[alex-ju]

What

Remove redundant import in the accordion component that was accidentally introduced in #1884.

We do however need it for the print stylesheet, as we have two applications requiring the components/print/accordion but not the component_support. This could potentially be fixed in future by removing component_support from print/_accordion and updating the suggested print imports to include component_support.

Why

This import resulted in a considerable increase in our render-blocking resources.

Visual Changes

No visual changes

[andysellick]

Looks sensible. Can you record here specifically what it is that this component needs from component_support?

[alex-ju]

Looks sensible. Can you record here specifically what it is that this component needs from component_support?

All the govuk-frontend mixins used in accordion 🙂

[chris-gds]

Ahh thanks for this @alex-ju! Do we want to use the govuk-frontend mixins in this way? 🤔

[alex-ju]

Ahh thanks for this @alex-ju! Do we want to use the govuk-frontend mixins in this way? 🤔

Yes, we want to use govuk-frontend mixins as much as possible as it helps with building components in a more consistent way, but these are already included by each app, so there's no need to import them into each component. The only different thing with this component is that the accordion print stylesheet requires the accordion component stylesheet (as the Design System doesn't ship print stylesheets at the moment) so we don't show buttons and other unnecessary elements when printing the page.