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

fix typos #1704

Closed
wants to merge 3 commits into from
Closed
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
8 changes: 4 additions & 4 deletions src/css/video-js.less
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,7 @@ Create your own skin at http://designer.videojs.com
// (This file uses LESS. Learn more at http://lesscss.org/)

// The base font size controls the size of everything, not just text. All
// diminensions use em-based sizes so that the scale along with the font size.
// dimensions use em-based sizes so that the scale along with the font size.
// Try increasing it to 20px and see what happens.
@base-font-size: 10px;
@touch-device-font-size: 15px;
Expand Down Expand Up @@ -60,7 +60,7 @@ Create your own skin at http://designer.videojs.com
/* SKIN
================================================================================
The main class name for all skin-specific styles. To make your own skin,
replace all occurances of 'vjs-default-skin' with a new name. Then add your new
replace all occurrences of 'vjs-default-skin' with a new name. Then add your new
skin name to your video tag instead of the default skin.
e.g. <video class="video-js my-skin-name">
*/
Expand Down Expand Up @@ -111,7 +111,7 @@ The control icons are from a custom font. Each icon corresponds to a character

/* Slider - used for Volume bar and Seek bar */
.vjs-default-skin .vjs-slider {
/* Replace browser focus hightlight with handle highlight *///
/* Replace browser focus highlight with handle highlight *///
outline: 0;
position: relative;
cursor: pointer;
Expand Down Expand Up @@ -855,7 +855,7 @@ control positioning and full window mode. **
/* Start with 10px for base font size so other dimensions can be em based and
easily calculable. */
font-size: @base-font-size;
/* Allow poster to be vertially aligned. */
/* Allow poster to be vertically aligned. */
vertical-align: middle;
/* display: table-cell; */ /*This works in Safari but not Firefox.*/

Expand Down
10 changes: 5 additions & 5 deletions src/js/component.js
Original file line number Diff line number Diff line change
Expand Up @@ -561,7 +561,7 @@ vjs.Component.prototype.buildCSSClass = function(){
*
* The benefit of using this over `vjs.on(otherElement, 'eventName', myFunc)`
* and `otherComponent.on('eventName', myFunc)` is that this way the listeners
* will be automatically cleaned up when either component is diposed.
* will be automatically cleaned up when either component is disposed.
* It will also bind myComponent as the context of myFunc.
*
* **NOTE**: When using this on elements in the page other than window
Expand Down Expand Up @@ -742,7 +742,7 @@ vjs.Component.prototype.isReady_;
*
* Allows for delaying ready. Override on a sub class prototype.
* If you set this.isReadyOnInitFinish_ it will affect all components.
* Specially used when waiting for the Flash player to asynchrnously load.
* Specially used when waiting for the Flash player to asynchronously load.
*
* @type {Boolean}
* @private
Expand All @@ -760,7 +760,7 @@ vjs.Component.prototype.readyQueue_;
/**
* Bind a listener to the component's ready state
*
* Different from event listeners in that if the ready event has already happend
* Different from event listeners in that if the ready event has already happened
* it will trigger the function immediately.
*
* @param {Function} fn Ready listener
Expand Down Expand Up @@ -886,7 +886,7 @@ vjs.Component.prototype.unlockShowing = function(){
/**
* Disable component by making it unshowable
*
* Currently private because we're movign towards more css-based states.
* Currently private because we're moving towards more css-based states.
* @private
*/
vjs.Component.prototype.disable = function(){
Expand Down Expand Up @@ -1024,7 +1024,7 @@ vjs.Component.prototype.onResize;
*
* This is used to support toggling the controls through a tap on the video.
*
* We're requireing them to be enabled because otherwise every component would
* We're requiring them to be enabled because otherwise every component would
* have this extra overhead unnecessarily, on mobile devices where extra
* overhead is especially bad.
* @private
Expand Down
12 changes: 6 additions & 6 deletions src/js/core-object.js
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
/**
* Core Object/Class for objects that use inheritance + contstructors
* Core Object/Class for objects that use inheritance + constructors
*
* To create a class that can be subclassed itself, extend the CoreObject class.
*
Expand Down Expand Up @@ -52,7 +52,7 @@
vjs.CoreObject = vjs['CoreObject'] = function(){};
// Manually exporting vjs['CoreObject'] here for Closure Compiler
// because of the use of the extend/create class methods
// If we didn't do this, those functions would get flattend to something like
// If we didn't do this, those functions would get flattened to something like
// `a = ...` and `this.prototype` would refer to the global object instead of
// CoreObject

Expand All @@ -78,10 +78,10 @@ vjs.CoreObject.extend = function(props){
// In Resig's simple class inheritance (previously used) the constructor
// is a function that calls `this.init.apply(arguments)`
// However that would prevent us from using `ParentObject.call(this);`
// in a Child constuctor because the `this` in `this.init`
// would still refer to the Child and cause an inifinite loop.
// in a Child constructor because the `this` in `this.init`
// would still refer to the Child and cause an infinite loop.
// We would instead have to do
// `ParentObject.prototype.init.apply(this, argumnents);`
// `ParentObject.prototype.init.apply(this, arguments);`
// Bleh. We're not creating a _super() function, so it's good to keep
// the parent constructor reference simple.
subObj = function(){
Expand Down Expand Up @@ -110,7 +110,7 @@ vjs.CoreObject.extend = function(props){
};

/**
* Create a new instace of this Object class
* Create a new instance of this Object class
*
* var myAnimal = Animal.create();
*
Expand Down
2 changes: 1 addition & 1 deletion src/js/events.js
Original file line number Diff line number Diff line change
Expand Up @@ -325,7 +325,7 @@ vjs.trigger = function(elem, event) {
* We've since updated to the latest version, but keeping this around
* for now just in case.
*/
// // Added in attion to book. Book code was broke.
// // Added in addition to book. Book code was broke.
// event = typeof event === 'object' ?
// event[vjs.expando] ?
// event :
Expand Down
6 changes: 3 additions & 3 deletions src/js/exports.js
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
/**
* @fileoverview Exports for Video.js.
* Exports are publically available variables.
* Exports are publicly available variables.
* All other variables (including function names) will probably
* be renamed by closure compiler.
*/
Expand All @@ -10,15 +10,15 @@
*
* vjs is the same as the goog var in Closure Library. It holds all variables
* used in Video.js development. Closure compiler will rename all variables,
* including class prototype functions, except those specifcally
* including class prototype functions, except those specifically
* exported (eports.js). Don't assume any function you can use in Video.js
* development will be available on window.videojs for use with other js.
*
* For example, vjs.trim is an internal function and will be renamed by compiler
* to something like 'a.b', or actually more likely 'a' removing it from
* a parent object.
*
* videojs is a var that helps bridge bewteen internal and external development.
* videojs is a var that helps bridge between internal and external development.
* Avoid using it over vjs when developing the Video.js core.
*
* _V_ is only external. It's just cute and short(er). Like jQuery === $.
Expand Down
2 changes: 1 addition & 1 deletion src/js/fullscreen-api.js
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@
var apiMap, specApi, browserApi, i;

/**
* Store the browser-specifc methods for the fullscreen API
* Store the browser-specific methods for the fullscreen API
* @type {Object|undefined}
* @private
*/
Expand Down
12 changes: 6 additions & 6 deletions src/js/lib.js
Original file line number Diff line number Diff line change
Expand Up @@ -117,7 +117,7 @@ vjs.obj.merge = function(obj1, obj2){
vjs.obj.deepMerge = function(obj1, obj2){
var key, val1, val2;

// make a copy of obj1 so we're not ovewriting original values.
// make a copy of obj1 so we're not overwriting original values.
// like prototype.options_ and all sub options objects
obj1 = vjs.obj.copy(obj1);

Expand Down Expand Up @@ -213,7 +213,7 @@ vjs.bind = function(context, fn, uid) {

/**
* Element Data Store. Allows for binding data to an element without putting it directly on the element.
* Ex. Event listneres are stored here.
* Ex. Event listeners are stored here.
* (also from jsninja.com, slightly modified and updated for closure compiler)
* @type {Object}
* @private
Expand Down Expand Up @@ -432,7 +432,7 @@ vjs.setElementAttributes = function(el, attributes){

/**
* Get an element's attribute values, as defined on the HTML tag
* Attributs are not the same as properties. They're defined on the tag
* Attributes are not the same as properties. They're defined on the tag
* or with setAttribute (which shouldn't be used with HTML)
* This will return true or false for boolean attributes.
* @param {Element} tag Element from which to get tag attributes
Expand Down Expand Up @@ -695,7 +695,7 @@ vjs.get = function(url, onSuccess, onError, withCredentials){
};

/**
* Add to local storage (may removeable)
* Add to local storage (may removable)
* @private
*/
vjs.setLocalStorage = function(key, value){
Expand All @@ -718,7 +718,7 @@ vjs.setLocalStorage = function(key, value){
};

/**
* Get abosolute version of relative URL. Used to tell flash correct URL.
* Get absolute version of relative URL. Used to tell flash correct URL.
* http://stackoverflow.com/questions/470832/getting-an-absolute-url-from-a-relative-one-ie6-issue
* @param {String} url URL to make absolute
* @return {String} Absolute URL
Expand Down Expand Up @@ -780,7 +780,7 @@ vjs.parseUrl = function(url) {
};

/**
* Log messags to the console and history based on the type of message
* Log messages to the console and history based on the type of message
*
* @param {String} type The type of message, or `null` for `log`
* @param {[type]} args The args to be passed to the log
Expand Down
4 changes: 2 additions & 2 deletions src/js/media/flash.js
Original file line number Diff line number Diff line change
Expand Up @@ -58,7 +58,7 @@ vjs.Flash = vjs.MediaTechController.extend({
// Merge default attributes with ones passed in
attributes = vjs.obj.merge({
'id': objId,
'name': objId, // Both ID and Name needed or swf to identifty itself
'name': objId, // Both ID and Name needed or swf to identify itself
'class': 'vjs-tech'
}, options['attributes'])
;
Expand Down Expand Up @@ -129,7 +129,7 @@ vjs.Flash.prototype.src = function(src){
this.setRtmpConnection(src.connection);
this.setRtmpStream(src.stream);
} else {
// Make sure source URL is abosolute.
// Make sure source URL is absolute.
src = vjs.getAbsoluteURL(src);
this.el_.vjs_src(src);
}
Expand Down
2 changes: 1 addition & 1 deletion src/js/media/media.js
Original file line number Diff line number Diff line change
Expand Up @@ -23,7 +23,7 @@ vjs.MediaTechController = vjs.Component.extend({
this.manualProgressOn();
}

// Manually track timeudpates in cases where the browser/flash player doesn't report it.
// Manually track timeupdates in cases where the browser/flash player doesn't report it.
if (!this['featuresTimeupdateEvents']) {
this.manualTimeUpdatesOn();
}
Expand Down
20 changes: 10 additions & 10 deletions src/js/player.js
Original file line number Diff line number Diff line change
Expand Up @@ -108,7 +108,7 @@ vjs.Player = vjs.Component.extend({
});

/**
* The players's stored language code
* The player's stored language code
*
* @type {String}
* @private
Expand All @@ -131,7 +131,7 @@ vjs.Player.prototype.language = function (languageCode) {
};

/**
* The players's stored language dictionary
* The player's stored language dictionary
*
* @type {Object}
* @private
Expand Down Expand Up @@ -315,7 +315,7 @@ vjs.Player.prototype.createEl = function(){

// /* Media Technology (tech)
// ================================================================================ */
// Load/Create an instance of playback technlogy including element and API methods
// Load/Create an instance of playback technology including element and API methods
// And append playback element in player div.
vjs.Player.prototype.loadTech = function(techName, source){

Expand Down Expand Up @@ -454,15 +454,15 @@ vjs.Player.prototype.onPlay = function(){
};

/**
* Fired whenever the media begins wating
* Fired whenever the media begins waiting
* @event waiting
*/
vjs.Player.prototype.onWaiting = function(){
this.addClass('vjs-waiting');
};

/**
* A handler for events that signal that waiting has eneded
* A handler for events that signal that waiting has ended
* which is not consistent between browsers. See #1351
* @private
*/
Expand Down Expand Up @@ -552,7 +552,7 @@ vjs.Player.prototype.onEnded = function(){
* @event durationchange
*/
vjs.Player.prototype.onDurationChange = function(){
// Allows for cacheing value instead of asking player each time.
// Allows for caching value instead of asking player each time.
// We need to get the techGet response and check for a value so we don't
// accidentally cause the stack to blow up.
var duration = this.techGet('duration');
Expand Down Expand Up @@ -710,7 +710,7 @@ vjs.Player.prototype.currentTime = function(seconds){
// cache last currentTime and return. default to 0 seconds
//
// Caching the currentTime is meant to prevent a massive amount of reads on the tech's
// currentTime when scrubbing, but may not provide much performace benefit afterall.
// currentTime when scrubbing, but may not provide much performance benefit afterall.
// Should be tested. Also something has to read the actual current time or the cache will
// never get updated.
return this.cache_.currentTime = (this.techGet('currentTime') || 0);
Expand All @@ -730,7 +730,7 @@ vjs.Player.prototype.currentTime = function(seconds){
vjs.Player.prototype.duration = function(seconds){
if (seconds !== undefined) {

// cache the last set value for optimiized scrubbing (esp. Flash)
// cache the last set value for optimized scrubbing (esp. Flash)
this.cache_.duration = parseFloat(seconds);

return this;
Expand Down Expand Up @@ -896,7 +896,7 @@ vjs.Player.prototype.muted = function(muted){
};

// Check if current tech can support native fullscreen
// (e.g. with built in controls lik iOS, so not our flash swf)
// (e.g. with built in controls like iOS, so not our flash swf)
vjs.Player.prototype.supportsFullScreen = function(){
return this.techGet('supportsFullScreen') || false;
};
Expand Down Expand Up @@ -967,7 +967,7 @@ vjs.Player.prototype.requestFullscreen = function(){

// Trigger fullscreenchange event after change
// We have to specifically add this each time, and remove
// when cancelling fullscreen. Otherwise if there's multiple
// when canceling fullscreen. Otherwise if there's multiple
// players on a page, they would all be reacting to the same fullscreen
// events
vjs.on(document, fsApi['fullscreenchange'], vjs.bind(this, function(e){
Expand Down
10 changes: 5 additions & 5 deletions src/js/tracks.js
Original file line number Diff line number Diff line change
@@ -1,8 +1,8 @@
/**
* @fileoverview Text Tracks
* Text tracks are tracks of timed text events.
* Captions - text displayed over the video for the hearing impared
* Subtitles - text displayed over the video for those who don't understand langauge in the video
* Captions - text displayed over the video for the hearing impaired
* Subtitles - text displayed over the video for those who don't understand language in the video
* Chapters - text displayed in a menu allowing the user to jump to particular points (chapters) in the video
* Descriptions (not supported yet) - audio descriptions that are read back to the user by a screen reading device
*/
Expand Down Expand Up @@ -55,8 +55,8 @@ vjs.Player.prototype.addTextTrack = function(kind, label, language, options){
tracks.push(track);

// If track.dflt() is set, start showing immediately
// TODO: Add a process to deterime the best track to show for the specific kind
// Incase there are mulitple defaulted tracks of the same kind
// TODO: Add a process to determine the best track to show for the specific kind
// In case there are multiple defaulted tracks of the same kind
// Or the user has a set preference of a specific language that should override the default
// Note: The setTimeout is a workaround because with the html5 tech, the player is 'ready'
// before it's child components (including the textTrackDisplay) have finished loading.
Expand Down Expand Up @@ -472,7 +472,7 @@ vjs.TextTrack.prototype.parseCues = function(srcContent) {
text = [];

// Loop until a blank line or end of lines
// Assumeing trim('') returns false for blank lines
// Assuming trim('') returns false for blank lines
while (lines[++i] && (line = vjs.trim(lines[i]))) {
text.push(line);
}
Expand Down