acejump / AceJump

🅰️ single character search, select, and jump
https://plugins.jetbrains.com/plugin/7086-acejump
GNU General Public License v3.0
1.23k stars 91 forks source link

Sometimes can not jump to word in version 3.8.15 #446

Closed JohnWickzzz closed 1 year ago

JohnWickzzz commented 1 year ago

Describe the bug I don't even know how to describe or reproduce this bug consistently. Like the title, I don't know when it will occur. eg: I want to jump to a word "List", after the shortcut 'ctrl-;', I enter key "L", like the image image but at this time, when I enter 'J', nothing happened, however I can jump to the 'G' word.

To Reproduce Steps to reproduce the behavior: like the desc, I can not reproduce this bug consistently. It just 'sometimes'

Expected behavior Can jump to any word.

Desktop (please complete the following information):

Additional context In AceJump version 3.8.14, everything is ok(maybe), because I don't know under which circumstances this phenomenon will occur either.

KienSetel commented 1 year ago

It's happening to me as well. I just updated my Android Studio to Giraffe and this bug really annoys me. Just like @buptlibo the circumstances are not consistent. But can easily reproduce though.

zhujiangzhou commented 1 year ago

I have also encountered this problem recently.

shawnsong commented 1 year ago

Having the same issue here.

luoxiao78 commented 1 year ago

same here, the inconsistency is very annoying. plz fix it.

zxeoc commented 1 year ago

same issue.

OS: Linux zx 6.4.12-arch1-1 IDEA: 2023.2.1 AceJump: 3.8.15

breandan commented 1 year ago

My bad! We recently tried to fix an issue where users would accidentally jump to off-screen tags, but then introduced a regression where visible tags are occasionally classified as off-screen and thus unavailable to select. Since this bug is difficult to reproduce, I am not 100% sure if d905909e31c40d7ededd9f1a422c6592641d2a8c will fix the issue, but it is borrowed directly from the platform implementation of Editor.calculateVisibleRange() so it should be more stable than the previous implementation. This should be available through an update from the plugin marketplace shortly. Please report back if you encounter this issue after upgrading to AceJump 3.8.16. Thanks!

JohnWickzzz commented 1 year ago

I think it works, thank you! @breandan

KienSetel commented 1 year ago

Thank you! @breandan , it works flawlessly for me so far!

tanmaychimurkar commented 1 year ago

Hi everyone, I had the same issue with 3.8.15, and I can also confirm that this has been fixed with 3.8.16!