cehfisher / a11y-style-guide

Accessibility (A11Y) Style Guide
http://a11y-style-guide.com/style-guide
MIT License
558 stars 61 forks source link

Missing asset called by all.css #221

Closed alexroseb closed 6 years ago

alexroseb commented 6 years ago

Issue

In dist/all.css, it references an asset "../images/px-video-sprite.png" that doesn't exist. This occurs on lines 3762, 3817, 3844, and 3869. No other lines reference the missing "../images" directory.

cehfisher commented 6 years ago

What component should be using the px-video-sprite.png? I'm wondering if that is just an artifact of something that was removed possibly?

alexroseb commented 6 years ago

The components are:

/* mute button */
.px-video-mute-btn-container label
/* buttons */
.px-video-controls button
.px-video-fullscreen-btn-container label 
/* captions button */
.px-video-captions-btn-container label 

and it always appears in the context background: no-repeat url("../images/px-video-sprite.png");

cehfisher commented 6 years ago

I'm not using this in any live components any more. Thanks for letting me know so I can remove it from the code. I'll keep this ticket open as a reminder :)