Bug 1994400 Comment 14 Edit History

Note: The actual edited comment in the bug view page will always show the original commenter’s name and original timestamp.

Original comment by
on 
Sorry for the delay. I looked into the issue today and seems like the problem is hidePopup() steals focus, which breaks Korean IME composition when typing in input fields.

@mardak,
wonder would it be acceptable to skip showing aiActionButton for the shortcut when user selects text inside input/textarea/contenteditable elements? This avoids the hidePopup() focus issue entirely. or the only way I could fix it was moving the panel offscreen with panel.moveTo(-9999, -9999) instead of calling hidePopup() which i don't want to do it.
Revision 1 by
on 
Sorry for the delay. I looked into the issue today and as Kagami Rosylight [:saschanaz] (they/them) mentioned  the problem is hidePopup() steals focus, which breaks Korean IME composition when typing in input fields.

@mardak,
wonder would it be acceptable to skip showing aiActionButton for the shortcut when user selects text inside input/textarea/contenteditable elements? This avoids the hidePopup() focus issue entirely. or the only way I could fix it was moving the panel offscreen with panel.moveTo(-9999, -9999) instead of calling hidePopup() which i don't want to do it.
Revision 2 by
on 
Sorry for the delay. I looked into the issue today and as Kagami Rosylight [:saschanaz] (they/them) mentioned  the problem is hidePopup() steals focus, which breaks Korean IME composition when typing in input fields.

@mardak,
wonder would it be acceptable to skip showing aiActionButton for the shortcut when user selects text inside input/textarea/contenteditable elements? This avoids the hidePopup() focus issue entirely. or the only way I could fix it was moving the panel offscreen with panel.moveTo(-9999, -9999) instead of calling hidePopup() which i don't want to do it. - actually we need the shortcut for the proofread...
Revision 3 by
on 
Sorry for the delay. I looked into the issue today and as Kagami Rosylight [:saschanaz] (they/them) mentioned  the problem is hidePopup() steals focus, which breaks Korean IME composition when typing in input fields.

@mardak,
wonder would it be acceptable to skip showing aiActionButton for the shortcut when user selects text inside input/textarea/contenteditable elements? This avoids the hidePopup() focus issue entirely. or the only way I could fix it was moving the panel offscreen with panel.moveTo(-9999, -9999) instead of calling hidePopup() which i don't want to do it. - actually we need the shortcut for the proofread in input/textarea...

Back to Bug 1994400 Comment 14