Skip to content

fix: render size#3

Merged
so1ve merged 3 commits into
so1ve:mainfrom
enpitsuLin:main
Jun 12, 2023
Merged

fix: render size#3
so1ve merged 3 commits into
so1ve:mainfrom
enpitsuLin:main

Conversation

@enpitsuLin

@enpitsuLin enpitsuLin commented Jun 12, 2023

Copy link
Copy Markdown
Contributor

No description provided.

@bolt-new-by-stackblitz

bolt-new-by-stackblitz Bot commented Jun 12, 2023

Copy link
Copy Markdown

Review PR in StackBlitz Codeflow Run & review this pull request in StackBlitz Codeflow.

@so1ve so1ve marked this pull request as draft June 12, 2023 09:35
@enpitsuLin

enpitsuLin commented Jun 12, 2023

Copy link
Copy Markdown
Contributor Author

8太对 每次都重新new了一个

@so1ve

so1ve commented Jun 12, 2023

Copy link
Copy Markdown
Owner

8太对 每次都重新new了一个

是这样（

@enpitsuLin

enpitsuLin commented Jun 12, 2023

Copy link
Copy Markdown
Contributor Author

8太对 每次都重新new了一个

是这样（

构造函数里把没用的响应式变量去掉就好了

@so1ve

so1ve commented Jun 12, 2023

Copy link
Copy Markdown
Owner

8太对 每次都重新new了一个

是这样（

构造函数里把没用的响应式变量去掉就好了

o.o?没听明白（

@so1ve so1ve marked this pull request as ready for review June 12, 2023 09:43
@so1ve

so1ve commented Jun 12, 2023

Copy link
Copy Markdown
Owner

佬！！！！！！！！！！你是我的神 😭

@enpitsuLin

enpitsuLin commented Jun 12, 2023

Copy link
Copy Markdown
Contributor Author

watchEffect 其实可以看做不用写deps的useEffect 抄react实现我都这样写的 不过要注意内部响应式变量

@so1ve so1ve merged commit 8158bb3 into so1ve:main Jun 12, 2023
@so1ve

so1ve commented Jun 12, 2023

Copy link
Copy Markdown
Owner

watchEffect 其实可以看做不用写deps的useEffect 抄react实现我都这样写的 不过要注意内部响应式变量

是这样，我一开始就是用的watchEffect，后来改成watch这个奇怪的bug依旧存在（

@enpitsuLin

enpitsuLin commented Jun 12, 2023

Copy link
Copy Markdown
Contributor Author

watchEffect 其实可以看做不用写deps的useEffect 抄react实现我都这样写的 不过要注意内部响应式变量

是这样，我一开始就是用的watchEffect，后来改成watch这个奇怪的bug依旧存在（

主要是watchPostEffect可以拿来当onMounted用 我记得是从大圣那里知道的 不过原来的问题应该是 SkinViewer内部被 ref 都包上了出现了奇怪的问题， vue 里的外部来的类实例最好还是shallowRef/shallowReactive浅层响应式

@so1ve

so1ve commented Jun 12, 2023

Copy link
Copy Markdown
Owner

主要是watchPostEffect可以拿来当onMounted用 我记得是从大圣那里知道的 不过原来的问题应该是 SkinViewer内部被 ref 都包上了出现了奇怪的问题， vue 里的外部来的类实例最好还是shallowRef/shallowReactive浅层响应式

学到了学到了，每天学习一点新知识

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@enpitsuLin @so1ve