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.

Sign up for GitHub

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

Native controls updates #2499

Closed
wants to merge 8 commits into from
Closed

Native controls updates #2499

Changes from 1 commit
Commits
Show all changes
8 commits
Select commit Hold shift + click to select a range
5d31bdf
Use native controls on iphone.
gkatsev Jul 16, 2015
1af6089
Update nativeControls functionality
gkatsev Aug 21, 2015
80c9ca9
Add an IS_NATIVE_ANDROID to browser object
gkatsev Aug 21, 2015
28509f4
Default to native controls for touch on iphones and native android de…
gkatsev Aug 21, 2015
ebf758f
Add tech controls listners when disabling native controls usage
gkatsev Aug 21, 2015
92a637a
increase css specificity
gkatsev Aug 21, 2015
693ae2e
remove techCalls from usingNativeControls method
gkatsev Aug 21, 2015
d9c0263
Add controls method to TechFaker
gkatsev Aug 21, 2015
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
Prev Previous commit
Next Next commit
remove techCalls from usingNativeControls method
  • Loading branch information
@gkatsev
gkatsev committed Aug 21, 2015
commit 693ae2eb199ecd84805b4f5a261911cc6f445e27
2 changes: 0 additions & 2 deletions src/js/player.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -1941,7 +1941,6 @@ class Player extends Component {
if (bool) {
this.removeTechControlsListeners();
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

make sure to remove these listeners if they got added, otherwise, it will mess things up.
Our custom controls will show up unnecessarily, and also do weird stuff like prevent the video from playing correctly (#2277)

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This wouldn't hurt as a code comment if you feel like adding it.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do we need to re-add the listeners if we swap back to non-native controls?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I guess we should also re-add them below.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Added.

this.addClass('vjs-using-native-controls');
this.techCall('setControls', true);

/**
* player is using the native device controls
Expand All @@ -1955,7 +1954,6 @@ class Player extends Component {
} else {
this.addTechControlsListeners();
this.removeClass('vjs-using-native-controls');
this.techCall('setControls', false);

/**
* player is using the custom HTML controls
Expand Down