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 text-document tab related issues #1320

Merged
merged 2 commits into from
Feb 12, 2025
Merged

Fix text-document tab related issues #1320

merged 2 commits into from
Feb 12, 2025

Conversation

nighca
Copy link
Collaborator

@nighca nighca commented Feb 10, 2025

close #1315, #1316.

Copy link

qiniu-prow bot commented Feb 10, 2025

Skipping CI for Draft Pull Request.
If you want CI signal for your change, please convert it to an actual PR.
You can still manually trigger a test run with /test all

@nighca nighca requested a review from Copilot February 10, 2025 10:04

Choose a reason for hiding this comment

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

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

@nighca nighca marked this pull request as ready for review February 11, 2025 00:53
@qiniu-ci
Copy link

This PR has been deployed to the preview environment. You can explore it using the preview URL.

Warning

Please note that deployments in the preview environment are temporary and will be automatically cleaned up after a certain period. Make sure to explore it before it is removed. For any questions, contact the Go+ Builder team.

@@ -94,23 +94,15 @@ export class ResourceReferenceController extends Emitter<{

this.addDisposer(
watch(
this.providerRef,
(provider) => {
() => [this.providerRef.value, this.ui.project.filesHash, this.ui.activeTextDocument] as const,
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

主要是增加了 activeTextDocument 变化时刷新 items 的逻辑

@aofei aofei merged commit 60b37bc into goplus:dev Feb 12, 2025
5 checks passed
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.

Scroll position lost in code-editor while switching among different documents
3 participants