Fix SASS import

[rechl]

Fixes #6195

Description

When compiling .scss with node-sass in an angular project. the compiler can not use the import.

So i would like to remove the relative import.

This issue appears since #6055
As an alternative couldn't we use this one?

@import "node_modules/videojs-font/scss/icons";

Requirements Checklist

• Feature implemented / Bug fixed
• If necessary, more likely in a feature request than a bug fix
  • Change has been verified in an actual browser (Chome, Firefox, IE)
  • Unit Tests updated or fixed
  • Docs/guides updated
  • Example created (starter template on JSBin)
• Reviewed by Two Core Contributors

[welcome]

💖 Thanks for opening this pull request! 💖

Things that will help get your PR across the finish line:

• Run npm run lint -- --errors locally to catch formatting errors earlier.
• Include tests when adding/changing behavior.
• Include screenshots and animated GIFs whenever possible.

We get a lot of pull requests on this repo, so please be patient and we will get back to you as soon as we can.

[tsi]

@sgotre why is this closed? is it fixed? we are experiencing a similar issue with 7.6.5/6

[rechl]

@tsi because it was a duplicate and the tests did not complete all green.
this issue had been discussed here #4076
and my PR seems to be incompatible with the authors build system dart-sass.

To get around this, we copied the problematic sass file into our project and fixed it there for us.

[tsi]

Tnx, I see, seems like this was fixed in 5.x and 6.x but not in master (7)
created a PR here #6326

[rechl]

@tsi i wasn't sure about the best solution and furthermore i found it a bit strange to rely on an "external" scss file. So maybe we could find a better/more stable solution which works with all build systems. This repo is getting a lot of PRs and it was easy to fix within our code... So i decided to close this PR in favor of others