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

Top row activate virtual keyboard #10

Open
neilswann80 opened this issue Mar 22, 2024 · 4 comments
Open

Top row activate virtual keyboard #10

neilswann80 opened this issue Mar 22, 2024 · 4 comments

Comments

@neilswann80
Copy link
Contributor

neilswann80 commented Mar 22, 2024

I've noticed a strange bug affecting the top row.

If you press the first square (1,1) it activates the first row of the virtual keyboard i.e. the left-cursor (previous clue) button.

If you press any other square on the first row it seems to activate the "current clue" section of the virtual keyboard thereby changing the solve direction.

If you press squares in the top-row whilst looking at the virtual keyboard you can see it's being triggered.

@nathonius
Copy link

nathonius commented Jun 8, 2024

I'm seeing the same thing. I'm guessing its something that's broken with recent koreader updates. I'm not familiar with the koreader source, but I'm going to take a crack at figuring out the issue here.

EDIT: After digging into it... I have no clue.

I think I can confirm it's not a simple off by one error though - it's not really the first row that has issues, as the problem area also seems to extend somewhat into the second row. This is easier to reproduce using the local emulator where you can precisely click.

@roygbyte
Copy link
Owner

@nathonius, thanks for the comment/reminder. This issue has plagued the plugin since as long as I can remember. So if it has something to do with a KOReader update, then that goes way back.

I poked around again with the behavior within the emulator. It appears that it's an area of the screen that produces this behavior, and not a range of cells. You can observe this by comparing the behavior between a regular crossword and a Sunday crossword. The Sunday crossword is more dense--more rows and columns. And the behavior extends into the fourth row (only by a bit! Some parts of the fourth row do not trigger the behavior). By comparison, in a regular crossword the behavior is triggered in the first two rows. Thus, I think it's a region of the screen, and not the cells.

What to do with this observation? I'm not sure yet!

@nathonius
Copy link

My instinct would be to compare the koreader navigation header tap area to the range that triggers this behavior. Nothing that I saw in the code building the layout looked suspect otherwise.

@roygbyte
Copy link
Owner

roygbyte commented Jun 12, 2024

Another thought (maybe a memory, actually) would be to log key presses from softkeyboard.lua. If I do recall, I believe taps in the area of the screen affecting this erroneous behavior may be triggering a keyboard event.

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

No branches or pull requests

3 participants