fix: the scss structure reimporting multiple times

[olzzon]

About the Contributor

this PR is made on the behalf of BBC

Type of Contribution

This is a:
Bug fix

Current Behavior

Currently the scss files is reimporting the same .scss files multiple times, ending up with a css styles stack of 144 :root elements.

New Behavior

The imports are done in the main.scss and removed in the sub scss files, as their already shared by the main.scss

Testing

It has been tested, going through the different components. But as all these 114 :root styles was identical, it shouldn't cause any issues.

• I have added one or more unit tests for this PR
• I have updated the relevant unit tests
• No unit test changes are needed for this PR

Affected areas

CSS - UI

Status

• PR is ready to be reviewed.
• The functionality has been tested by the author.
• Relevant unit tests has been added / updated.
• Relevant documentation (code comments, system documentation) has been added / updated.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 60.41%. Comparing base (1d87ef1) to head (8b926ac).
Report is 232 commits behind head on release52.

Additional details and impacted files

@@              Coverage Diff              @@
##           release52    #1335      +/-   ##
=============================================
- Coverage      60.68%   60.41%   -0.28%     
=============================================
  Files            459      461       +2     
  Lines          78126    79486    +1360     
  Branches        3599     4492     +893     
=============================================
+ Hits           47414    48018     +604     
- Misses         30516    31343     +827     
+ Partials         196      125      -71     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.