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

Bugfix: change of selectedId didn't changne selected transformer box #95

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

ivankorol01
Copy link

Change of selectedId didn't changne selected transformer box

When change of selectedId was triggered programmatically by the application logic: only the
selection of the box and display of comment box took place, Transformer box was always stuck on
previously selected annotation. This did not happen when selection was changed by selecting new box
with the mouse, because Transformer was set in that scenario in DefaultAnnotationState.onMouseDown.
This was misleading the user. I added Next button in the storybook to reproduce the faulty behavior.

…gne selected transformer box

When change of selectedId was triggered programmatically by the application logic: only the
selection of the box and display of comment box took place, Transformer box was always stuck on
previously selected annotation. This did not happen when selection was changed by selecting new box
with the mouse, because Transformer was set in that scenario in DefaultAnnotationState.onMouseDown.
This was misleading the user. I added Next button in the storybook to reproduce the faulty behavior.
Copy link

@accesslint accesslint bot left a comment

Choose a reason for hiding this comment

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

There are accessibility issues in these changes.

stories/index.stories.tsx Show resolved Hide resolved
@ivankorol01
Copy link
Author

If possible, please prioritise this pull request. It's a simple fix and solves important issue for me :)
Thank you!

@ivankorol01 ivankorol01 closed this Jan 9, 2022
@ivankorol01 ivankorol01 reopened this Jan 9, 2022
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.

1 participant