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

Configurable alpha parameter for webgl construction #1774

Merged
merged 1 commit into from Jun 30, 2017

Conversation

pandamicro
Copy link
Contributor

Re: cocos-creator/fireball#5965

Changes proposed in this pull request:

  • Configurable alpha parameter for webgl construction

Makes sure these boxes are checked before submitting your PR - thank you!

  • If your pull request has gone "stale", you should rebase your work on top of the latest version of the upstream branch.
  • If your commit history is full of small, unimportant commits (such as "fix pep8" or "update tests"), squash your commits down to a few, or one, discreet changesets before submitting a pull request.
  • To official teams:
    • Check that your javascript is following our style guide and end files with a newline
    • Document new code with comments in source code based on API Docs
    • Make sure any runtime log information in cc.log , cc.error, cc.warn or cc.assert has been moved into DebugInfos.js with an ID

@cocos-creator/engine-admins

* Web only.
* !#zh
* 用于设置 Canvas 背景是否支持 alpha 通道,默认为 false,这样可以有更高的性能表现。
* 如果你希望 Canvas 背景是透明的,并显示背后的其他 DOM 元素,你可以在 `cc.game.run` 之前将这个值设为 true。
Copy link
Contributor

Choose a reason for hiding this comment

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

建议加上:“但是这样有可能导致叠加一些混合模式时渲染异常”

Copy link
Contributor Author

Choose a reason for hiding this comment

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

你也没办法说清楚是什么情况下异常,我觉得还是不要加了,反正 alpha 为 false 的情况表现和 true 就是不同的,具体怎么不同都是跟 webgl 有关系,跟我们引擎没关系

Copy link
Contributor

Choose a reason for hiding this comment

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

@nantas nantas merged commit 5f2118d into cocos:v1.5 Jun 30, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants