Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add an Audio Description icon to the V5 icon font #2165

Closed
OwenEdwards opened this issue May 18, 2015 · 10 comments
Closed

Add an Audio Description icon to the V5 icon font #2165

OwenEdwards opened this issue May 18, 2015 · 10 comments

Comments

@OwenEdwards
Copy link
Member

Since support for text tracks is already present in V4.12, including partial support for 'description' tracks, at some point in V5 we'll be adding support for Audio Description (through a text track, and potentially through other means).

To prepare for this, can we add an audio description icon to the font, similar to the CC icon, based on the one here:

http://www.ryerson.ca/accessibility/tools/access_symbols.html

and:

http://www.acb.org/adp/ad.html#symbols

I don't know how to create the fonts, otherwise I'd do a PR myself. @mmcc?

@mmcc
Copy link
Member

mmcc commented May 18, 2015

@OwenEdwards this seems reasonable to me! Our current font is a subset of the Material UI font along with at least one custom icon of ours.

Basically the work flow right now would be upload the current SVG icon font to Icomoon, pull in the new icon, and re-export. I started to try and do some work around just having an automated build process with a folder of SVGs, but as it turns out, fonts are hard.

@OwenEdwards
Copy link
Member Author

Thanks @mmcc! I can get the basic AD icon in there (see my plugin to support switching a video with description already added, which addressed #1875. But the icon doesn't seem to be the same size as the ones in the main font, so combining it into the main font doesn't work. Any suggestions? Like you say, fonts are hard!

@OwenEdwards OwenEdwards added this to the WCAG 2.0 Compliance milestone May 19, 2015
@mmcc
Copy link
Member

mmcc commented May 19, 2015

Hrm...this might get a little tricky... Best case scenario is that we can get an SVG and upload that to Icomoon, because then they'll at least do most of the work around resizing for us. Unfortunately, I don't see an SVG version of those icons anywhere, so we might have to do it ourselves :(

I can take a wack at it with Sketch. I'm not a pro, but I've at least got some basic icon guides that I can use.

@OwenEdwards
Copy link
Member Author

I can get you an SVG of the icon! I'll email it...

(EDIT): For the purposes of this Issue thread, the highest quality icon for Audio Description seems to be: Adobe Illustrator (AI) Black on White from the ACB Audio Description Project page. This can be converted to SVG, but needs some editing to make it a square icon.

@mmcc
Copy link
Member

mmcc commented Jun 3, 2015

Looks like the SVG you gave me isn't playing nice with our build tool icon :(

@mmcc
Copy link
Member

mmcc commented Jun 3, 2015

I'll see if I can fix it and let you know

@OwenEdwards
Copy link
Member Author

Yikes! Can you use the original ad.ai from Adobe Illustrator (AI) Black on White, and export as SVG from Adobe Illustrator, or convert it at https://cloudconvert.com/ai-to-svg?

@mmcc
Copy link
Member

mmcc commented Jun 3, 2015

I had to fix it manually in Sketch, but it's been added to the new videojs-font project! It looks a tiny bit pixelated, so if someone else with more SVG-foo than me wants to take a stab, feel free.

@mmcc
Copy link
Member

mmcc commented Jun 3, 2015

@OwenEdwards There's a pull request open if you want to take a look: videojs/font#1

@mmcc
Copy link
Member

mmcc commented Jun 4, 2015

This has been pulled in, so an npm install followed by grunt should do the trick.

@mmcc mmcc closed this as completed Jun 4, 2015
@github-actions github-actions bot locked as resolved and limited conversation to collaborators May 27, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

2 participants