Skip to content

🐛 Fix selection of syllable after edit modal.#853

Merged
yinanazhou merged 2 commits into
developfrom
selectbbox-modal-fix
Jun 2, 2022
Merged

🐛 Fix selection of syllable after edit modal.#853
yinanazhou merged 2 commits into
developfrom
selectbbox-modal-fix

Conversation

@jinh0

@jinh0 jinh0 commented May 30, 2022

Copy link
Copy Markdown
Contributor

Fixes #830 .

♻️ Remove console log and extract variable.

🐛 Fix empty bbox selected.

🐛 Fix keyboard shortcuts not working after hiding modal (w/ David).

♻️ Remove console log and extract variable.

🐛 Fix empty bbox selected.

🐛 Fix keyboard shortcuts not working after hiding modal (w/ David).
@jinh0 jinh0 changed the title 🐛 Fix selection of syllable after edit. 🐛 Fix selection of syllable after edit modal. May 30, 2022

@yinanazhou yinanazhou left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I noticed that when one bbox is selected, if the user clicks on the text to select another bbox, the previous bbox is selected instead of the new one in the background. As shown in the figure below, I was editing the text for sur, but lem is selected. Is there a way to fix this?
image

Also, could you please delete the console.log() in ModalWindow.ts?

@yinanazhou yinanazhou merged commit cb315ce into develop Jun 2, 2022
@cadagong cadagong deleted the selectbbox-modal-fix branch June 16, 2022 16:44
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