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

set webAudio volume immediately #4767

Merged
merged 1 commit into from Jul 3, 2019

Conversation

@umbrellaPP
Copy link
Contributor

commented Jul 1, 2019

Re: cocos-creator/2d-tasks#1582

changeLog:

  • 修复 webAudio 设置音量,没有立即生效的问题

相关论坛反馈:https://forum.cocos.com/t/creator-h5-demo/79989

@umbrellaPP

This comment has been minimized.

Copy link
Contributor Author

commented Jul 1, 2019

麻烦 @VisualSJ 有空看看这个修复有没有问题

@umbrellaPP

This comment has been minimized.

Copy link
Contributor Author

commented Jul 2, 2019

关联之前的修复:781efe4

可能需要 @VisualSJ 解释下,为什么之前需要 0.01 的延时

@umbrellaPP

This comment has been minimized.

Copy link
Contributor Author

commented Jul 3, 2019

之前设置 0.01 是为了解决一些浏览器上的兼容性问题,现在不需要了

@knoxHuang

This comment has been minimized.

Copy link
Collaborator

commented Jul 3, 2019

那之前需要兼容的浏览器上是否有问题呢?

@umbrellaPP

This comment has been minimized.

Copy link
Contributor Author

commented Jul 3, 2019

我在几个主流的 UC chrome qq 微信浏览器上测试,没什么问题

@jareguo jareguo merged commit 979d137 into cocos-creator:v2.1.2 Jul 3, 2019

1 check passed

ci/circleci: test Your tests passed on CircleCI!
Details
@jareguo

This comment has been minimized.

Copy link
Contributor

commented Jul 3, 2019

蛋疼,以后大家都注意,千万别在代码里出现这种 magic number,要嘛就写注释说明

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants
You can’t perform that action at this time.