Updated poster to use CSS styles to hide instead of show/hide methods. fixes #1567 #1568
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Member
heff
commented
Oct 8, 2014
heff
commented
- Poster will now reshow itself when a poster source is set after a non-source
- Relying on CSS instead of show/hide methods now
- Made it so empty strings would hide the poster image element
- Updated a few component methods that made it annoying to create a mock player (@dmlap)
- Refactored poster image tests
| this.id_ = options['id'] || (options['el'] && options['el']['id']); | ||
|
|
||
| // If there was no ID from the options, generate one | ||
| if (!this.id_) { |
|
LGTM |
|
Ok, just need to deal with |
…fixes videojs#1567 Also, refactored a lot of it.
|
@gkatsev fyi, I used |
|
tsk, tsk. :P |
|
ha :) |
jgubman
added a commit
to jgubman/video.js
that referenced
this pull request
Oct 29, 2014
* 'stable' of https://github.com/videojs/video.js: (22 commits) Added line to changelog for 4.10.1 Release 4.10.1 Removed alert... Release 4.10.0 @heff turned on the custom html controls for touch devices. closes videojs#1617 updated options to include sans-children, and included self-hosted font blurb added CORS information to tracks guide. Fixes videojs#837 doc updates + readme quick start updated options guide to include components @heff Added the ability to set options for child components directly in the parent options. closes videojs#1599 @DevGavin added a Simplified Chinese translation. closes videojs#1593 @mmcc fixed an issue with the VolumeButton assuming it was vertical by default. closes videojs#1592 @heff enhanced the event listener API to allow for auto-cleanup of listeners on other componenets and elements. closes videojs#1588 @mmcc fixed an issue where errors on source tags could get missed. closes videojs#1575 Added doc for remaining time and removed onWaitEnd doc maybe actually check for keyLocation Update languages.md @heff updated the poster to use CSS styles to display; fixed the poster not showing if not originally set. closes videojs#1568 Release 4.9.1 Bumped to videojs-swf v4.5.1 to fix a data sanitization issue. closes videojs#1587 ...
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.