Add AirPlay button to overflow menu

[avelad]

Resolves #1003

[michellezhuogg]

LGTM.

[theodab]

We also maintain a list of UI buttons in our docs, in ui-customization.md. You should list this button there.

[avelad]

@theodab @michellezhuogg Is there anything else I can do? Do you need any more changes?

[avelad]

I updated the PR with the change of button name

[theodab]

LGTM. I'll run the build bot on this.

[shaka-bot]

Test Failure:

Generating Closure dependencies...
Linting JavaScript...
Linting CSS...
Linting HTML...

Config loaded: /var/lib/jenkins/workspace/Manual PR Test (local-tests)/.htmlhintrc

Config loaded: /var/lib/jenkins/workspace/Manual PR Test (local-tests)/.htmlhintrc

Config loaded: /var/lib/jenkins/workspace/Manual PR Test (local-tests)/.htmlhintrc

Scanned 3 files, no errors found (40 ms).
Checking that the build files are complete...
Checking for common misspellings...
Checking correct usage of eslint-disable...
Checking the tests for type errors...
No changes detected, skipping. Use --force to override.
Building the docs...
�[31mSyntaxError: The "url" argument must be of type string. Received type object�[39m�[31m in �[39m/var/lib/jenkins/workspace/Manual PR Test (local-tests)/ui/controls.less�[90m on line 17, column 1:�[39m
�[90m16 // NOTE: Working around google/material-design-icons#958�[39m
17 �[7m�[31m�[1m@�[22mimport (css, inline) "https://fonts.googleapis.com/icon?family=Material+Icons+Round";�[39m�[27m
�[90m18 �[39m�[0m�[0m

Externs generation failed
END-BUILD: FAILURE
Build step 'Execute shell' marked build as failure

[shaka-bot]

Test Failure:

Generating Closure dependencies...
Linting JavaScript...
Linting CSS...
Linting HTML...

Config loaded: /var/lib/jenkins/workspace/Manual PR Test (local-tests)/.htmlhintrc

Config loaded: /var/lib/jenkins/workspace/Manual PR Test (local-tests)/.htmlhintrc

Config loaded: /var/lib/jenkins/workspace/Manual PR Test (local-tests)/.htmlhintrc

Scanned 3 files, no errors found (55 ms).
Checking that the build files are complete...
Checking for common misspellings...
Checking correct usage of eslint-disable...
Checking the tests for type errors...
No changes detected, skipping. Use --force to override.
Building the docs...
�[31mSyntaxError: The "url" argument must be of type string. Received type object�[39m�[31m in �[39m/var/lib/jenkins/workspace/Manual PR Test (local-tests)/ui/controls.less�[90m on line 17, column 1:�[39m
�[90m16 // NOTE: Working around google/material-design-icons#958�[39m
17 �[7m�[31m�[1m@�[22mimport (css, inline) "https://fonts.googleapis.com/icon?family=Material+Icons+Round";�[39m�[27m
�[90m18 �[39m�[0m�[0m

Externs generation failed
END-BUILD: FAILURE
Build step 'Execute shell' marked build as failure

[theodab]

Hm. I see the same test failure on other CLs that have been tested today, so this isn't your PR's fault.
I think the MDL people might have messed up the CSS they are hosting, or something like that?
I tried doing a local test run and I didn't see any issues, anyway. I think this is safe.

[theodab]

Thanks for your contributions!

[theodab]

Whoops, I added the semantic versioning tag into the commit message, rather than the title.

[IvarssonAndreas]

Hi!
What is the status of this issue?
What I can tell the contribution is merged but not included in any release?

[theodab]

This pull request has been merged into master. You can see it on the commit log: fac2913
It hasn't been put into a release yet. As you can see from the associated issue, #1003, it is currently slated to go into the upcoming v3.1 release.

[IvarssonAndreas]

Thank you!