🐛 do not ignore @import rules pointing to inaccessible stylesheets

[BenoitZugmeyer]

Motivation

We follow @import rules and inline their content so the player can uses its strategy to emulate :hover states. But if the external stylesheet is inaccessible, we should still take the @import rule into account, so the external stylesheet can be loaded in the player.

Changes

If we fail to get the external stylesheet rule, use the @import rule cssText as a fallback

Testing

• Local
• Staging
• Unit
• End to end

---

I have gone over the contributing documentation.

[BenoitZugmeyer]

/to-staging

[codecov-commenter]

Codecov Report

Merging #2398 (465a7bd) into main (ef8f6ca) will increase coverage by 0.01%.
Report is 2 commits behind head on main.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main    #2398      +/-   ##
==========================================
+ Coverage   93.99%   94.01%   +0.01%     
==========================================
  Files         212      212              
  Lines        6298     6299       +1     
  Branches     1405     1405              
==========================================
+ Hits         5920     5922       +2     
+ Misses        378      377       -1     

Files Changed	Coverage Δ
...domain/record/serialization/serializeAttributes.ts	98.64% <100.00%> (+1.38%)	⬆️

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

[dd-devflow]

🚂 Branch Integration: starting soon, merge in < 7m

commit 98431dde7e will soon be integrated into staging-35.

This build is going to start soon! (estimated merge in less than 7m)

you can cancel this operation by commenting your pull request with /to-staging -c!

[dd-devflow]

🚂 Branch Integration: this commit was successfully integrated

Commit 98431dde7e has been merged into staging-35 in merge commit 2c2997766d.

Check out the triggered pipeline on Gitlab 🦊

[bcaudan]

💬 suggestion: ‏a comment here could be helpful as well