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

Bug select few annotations in one item navigate to another item breaks #511

Open
wants to merge 5 commits into
base: main-variants
Choose a base branch
from

Conversation

orlinmalkja
Copy link
Contributor

No description provided.

@orlinmalkja orlinmalkja self-assigned this Oct 25, 2024
@orlinmalkja
Copy link
Contributor Author

@paulpestov the last two commits are the contribution of this PR. The others are part of the other PR.



// when target is null -> we switched to another item - we don't continue and change the level of highlight of the text
if (target) {
Copy link
Collaborator

Choose a reason for hiding this comment

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

if (!target) return ...But we have to come back here after the release.

@paulpestov
Copy link
Collaborator

Please don't add unnecessary commit to PRs anymore, you can do the merge afterwards. Look at the main-variants branch, it has now 2 duplicated 'update configs' commits.

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