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 the problem when the MeshBuffer is switched on the native platform. #1935

Open
wants to merge 1 commit into
base: v2.3.0-release
from

Conversation

@caryliu1999
Copy link
Contributor

caryliu1999 commented Nov 15, 2019

问题反馈:

https://forum.cocos.org/t/v2-20-mask/85523/13?u=cary

由于Graphics绘制顶点数量过多导致MeshBuffer切换,之前MeshBuffer切换时在原生平台存在一些问题,导致新的数据不会被绘制。

修改说明:

当MeshBuffer顶点数据超过65535时,原生层没有同步提交并切换顶点索引数据缓冲,这里更新同步Js层的处理。

关联PR: cocos-creator-packages/jsb-adapter#206

@holycanvas

This comment has been minimized.

Copy link
Contributor

holycanvas commented Nov 18, 2019

要不下个版本吧,看起来改动不下呀

@sunnylanwanjun

This comment has been minimized.

Copy link
Contributor

sunnylanwanjun commented Nov 18, 2019

嗯,我也觉得最好下个版本,这个改动需要做挺多边界测试的,有点慌

@holycanvas holycanvas changed the base branch from v2.2.1-release to v2.3.0 Nov 29, 2019
@holycanvas

This comment has been minimized.

Copy link
Contributor

holycanvas commented Jan 10, 2020

冲突了,另外这个最好测试下

@cocos-robot cocos-robot changed the base branch from v2.3.0 to v2.3.0-release Jan 13, 2020
Copy link
Contributor

cocos-robot left a comment

@caryliu1999, v2.3.0 branch will be deleted, so we edited the base branch to v2.3.0-release, or this PR will be killed by GitHub.
Please review the commits history to ensure that the PR does not polluted by unneeded commits from your origin branch.
If you need to merge to other branch, you can first click the Edit button on the right side of the PR title, then switch the base branch.
If necessary, welcome to resubmit a new PR. Thanks!

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