Skip to content

Commit f0a8f4a

Browse files
authored
Merge pull request #83 from contently/PETA-1335-with-jquery
Adds jquery as a dependency instead of a peer-dependency so it can be…
2 parents 2a8ee11 + 9a14c3b commit f0a8f4a

13 files changed

+18105
-3
lines changed

.tool-versions

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1 +1,3 @@
11
nodejs 10.15.2
2+
python 2.7.13
3+
yarn 1.22.19

package-lock.json

Lines changed: 18085 additions & 0 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

package.json

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
{
22
"name": "@contently/videojs-annotation-comments",
3-
"version": "2.0.1",
3+
"version": "2.1.1",
44
"description": "A plugin for video.js to add support for timeline moment/range comments and annotations",
55
"repository": "[email protected]:contently/videojs-annotation-comments.git",
66
"main": "build/videojs-annotation-comments.cjs.js",
@@ -23,8 +23,8 @@
2323
"@semantic-release/git": "^7.0.18",
2424
"babel-core": "^6.26.3",
2525
"babel-eslint": "^10.1.0",
26-
"babel-plugin-transform-object-rest-spread": "^6.26.0",
2726
"babel-plugin-transform-es2015-modules-simple-commonjs": "^0.3.0",
27+
"babel-plugin-transform-object-rest-spread": "^6.26.0",
2828
"babel-preset-es2015": "^6.24.1",
2929
"babel-preset-es2015-script": "^1.1.0",
3030
"babelify": "^7.3.0",
@@ -100,6 +100,7 @@
100100
]
101101
},
102102
"dependencies": {
103+
"jquery": "^3.6.3",
103104
"mitt": "^1.2.0"
104105
}
105106
}

src/js/annotation_comments.js

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -3,6 +3,8 @@
33
Can be registered to a videojs instance as a plugin
44
*/
55

6+
const $ = require('jquery');
7+
68
module.exports = videojs => {
79
require('./lib/polyfills');
810

src/js/components/comment.js

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -3,6 +3,7 @@
33
*/
44

55
const moment = require('moment');
6+
const $ = require('jquery');
67
const PlayerUIComponent = require('./../lib/player_ui_component');
78
const Utils = require('./../lib/utils');
89

src/js/components/controls.js

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -3,6 +3,7 @@
33
including all functionality to add new annotations
44
*/
55

6+
const $ = require('jquery');
67
const PlayerUIComponent = require('./../lib/player_ui_component');
78
const Utils = require('./../lib/utils');
89
const DraggableMarker = require('./draggable_marker.js');

src/js/components/draggable_marker.js

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -3,6 +3,7 @@
33
as drag occurs
44
*/
55

6+
const $ = require('jquery');
67
const Marker = require('./marker');
78
const Utils = require('./../lib/utils');
89

src/js/components/marker.js

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -2,6 +2,7 @@
22
Component for a timeline marker with capabilities to render on timeline, including tooltip for comment
33
*/
44

5+
const $ = require('jquery');
56
const PlayerUIComponent = require('./../lib/player_ui_component');
67
const Utils = require('./../lib/utils');
78

src/js/components/player_button.js

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -2,6 +2,7 @@
22
Component main 'annotation toggle' button in the player controls, including notifier for # annotations
33
*/
44

5+
const $ = require('jquery');
56
const PlayerUIComponent = require('./../lib/player_ui_component');
67

78
const templateName = 'player_button';

src/js/components/selectable_shape.js

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -2,6 +2,7 @@
22
Component for a shape that can be drug/sized on top of the video while adding a new annotation
33
*/
44

5+
const $ = require('jquery');
56
const Shape = require('./shape');
67
const Utils = require('./../lib/utils');
78

src/js/components/shape.js

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -2,6 +2,7 @@
22
Component for managing a shape (i.e. box drawn on the player) for an annotation
33
*/
44

5+
const $ = require('jquery');
56
const PlayerUIComponent = require('./../lib/player_ui_component');
67

78
module.exports = class Shape extends PlayerUIComponent {

src/js/lib/player_ui_component.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@
22
Base class all player UI components interit from - it includes lots of helper functions (to get reference to
33
the player $el, various classes/helpers, template rendering, etc)
44
*/
5-
5+
const $ = require('jquery');
66
const Handlebars = require('handlebars/runtime');
77
const PlayerComponent = require('./player_component');
88
const { templates } = require('../compiled/templates');

yarn.lock

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -5565,6 +5565,11 @@ java-properties@^1.0.0:
55655565
resolved "https://registry.npmjs.org/java-properties/-/java-properties-1.0.2.tgz#ccd1fa73907438a5b5c38982269d0e771fe78211"
55665566
integrity sha512-qjdpeo2yKlYTH7nFdK0vbZWuTCesk4o63v5iVOlhMQPfuIZQfW/HI35SjfhA+4qpg36rnFSvUK5b1m+ckIblQQ==
55675567

5568+
jquery@^3.6.3:
5569+
version "3.6.3"
5570+
resolved "https://registry.yarnpkg.com/jquery/-/jquery-3.6.3.tgz#23ed2ffed8a19e048814f13391a19afcdba160e6"
5571+
integrity sha512-bZ5Sy3YzKo9Fyc8wH2iIQK4JImJ6R0GWI9kL1/k7Z91ZBNgkRXE6U0JfHIizZbort8ZunhSI3jw9I6253ahKfg==
5572+
55685573
js-base64@^2.1.8:
55695574
version "2.5.2"
55705575
resolved "https://registry.npmjs.org/js-base64/-/js-base64-2.5.2.tgz#313b6274dda718f714d00b3330bbae6e38e90209"

0 commit comments

Comments
 (0)