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: exit fullscreen model dup error #96

Merged
merged 2 commits into from
Mar 14, 2024

Conversation

yangxiaolang
Copy link
Contributor

@yangxiaolang yangxiaolang commented Mar 14, 2024

aui-code-editor 使用中,全屏查看后退出全屏,使用 modelUri 的实例会因为 model 已存在而创建 model 失败,导致编辑器无法正常使用
在对话框动画效果实现之前应该比较难触发

直接返回已有的 model ,里面的 value 会丢,手动 setValue 似乎也不行;怀疑是 model 的 editor 实例也就是对话框里的那个销毁了,model 返回给新实例后也被回收了,才有的退出全屏后有内容然后又没了
所以每次创建新的,都会尝试 dispose 已有的

2024-03-07.16-47-16.mp4

Copy link

changeset-bot bot commented Mar 14, 2024

🦋 Changeset detected

Latest commit: 447b47c

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
ng-monaco-editor Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

Copy link

codesandbox-ci bot commented Mar 14, 2024

This pull request is automatically built and testable in CodeSandbox.

To see build info of the built libraries, click here or the icon next to each commit SHA.

@yinshuxun
Copy link
Contributor

加上 changeset 发布个新版本吧

@yangxiaolang yangxiaolang merged commit a2d430e into master Mar 14, 2024
5 checks passed
@yangxiaolang yangxiaolang deleted the fix/exit-fullscreen-model-dup branch March 14, 2024 02:33
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.

2 participants