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

Notepad++: braille doesn't move to the next line if last line is empty #17430

Closed
nvdaes opened this issue Nov 23, 2024 · 10 comments · Fixed by #17431
Closed

Notepad++: braille doesn't move to the next line if last line is empty #17430

nvdaes opened this issue Nov 23, 2024 · 10 comments · Fixed by #17431
Labels
app/scintilla Scintilla edit controls bug component/braille p3 https://github.com/nvaccess/nvda/blob/master/projectDocs/issues/triage.md#priority triaged Has been triaged, issue is waiting for implementation.
Milestone

Comments

@nvdaes
Copy link
Collaborator

nvdaes commented Nov 23, 2024

Steps to reproduce:

  1. Open Notepad++.
  2. In the opened document, type several lines.
  3. Type enter to add an empty line.
  4. Press control+end to move the system cursor to the end of the document.
  5. Press up arrow to move the cursor to the previous line.
  6. Try to run the next line command in a braille display.

Actual behavior:

The cursor is not moved to the last line of the document

Expected behavior:

The cursor should be moved to the last line of the document.

NVDA logs, crash dumps and other attachments:

System configuration

NVDA installed/portable/running from source:

Running from source

NVDA version:

source-master-c32e0e7 x86

Windows version:

Windows 11 23H2 (10.0.22631) workstation AMD64

Name and version of other software in use when reproducing the issue:

Notepad++ 7.81

Other information about your system:

Other questions

Does the issue still occur after restarting your computer?

Yes

Have you tried any other versions of NVDA? If so, please report their behaviors.

NVDA 2024.4.1, with same behavior.

If NVDA add-ons are disabled, is your problem still occurring?

Yes

Does the issue still occur after you run the COM Registration Fixing Tool in NVDA's tools menu?

Yes

nvda.log

@nvdaes nvdaes added bug component/braille p3 https://github.com/nvaccess/nvda/blob/master/projectDocs/issues/triage.md#priority app/scintilla Scintilla edit controls labels Nov 23, 2024
@nvdaes
Copy link
Collaborator Author

nvdaes commented Nov 23, 2024

@burmancomp , you may be interested in this issue.
I think that we may fix this if we expand the textInfo to the line before collapsing to end.
I've tested it locally and seems to work.

@burmancomp
Copy link
Contributor

Thanks @nvdaes for notifying me. I am not unfortunately notepad++ user but @irrah68 could possibly be interested in testing this.

@irrah68
Copy link

irrah68 commented Nov 23, 2024

I can reproduce this with the latest Alpha version of NVDA and Notepad++ Version 8.6.9.

@nvdaes
Copy link
Collaborator Author

nvdaes commented Nov 23, 2024

@irrah68 , if you want, please test pr #17431

@irrah68
Copy link

irrah68 commented Nov 24, 2024

I tested this pr, and it seems to work correctly. I can navigate to hte empty line by using braille display's navigation keys.

But I found another problem in this pr version. I tested with the latest Alpha and 2024.4.rc1 and they don't have this problem at all.

  1. Type a line containing some text and press enter.
  2. Add only one character.
  3. Press the up arrow key and home so that you are on the first character of the first line.
  4. Press Shift+End to select the content of the line.
  5. Start pressing Nvda+Alt+Home and Nvda+Alt+End. You'll hear the first character of the selected line and after pressing Nvda+Alt+End the only character of the next line, though it is not selected.

@nvdaes
Copy link
Collaborator Author

nvdaes commented Nov 24, 2024

Thanks @irrah68 . Now we don't expand to line before collapsing to end if selection is not collapsed.
Feel free to test this PR again.

@irrah68
Copy link

irrah68 commented Nov 24, 2024 via email

@nvdaes
Copy link
Collaborator Author

nvdaes commented Nov 24, 2024

@irrah68 wrote:

But there is a difference when selecting a line in Notepad++ and othe editors like Windows Notepad

Thanks @irrah68 .
I think that this is not related to this PR, so I'll mark it as ready for review.
In my local tests, I tried to change the length of the line as considered by NVDA in Notepad++, but this produced bugs, so I think that we can work on this PR and, if needed, more issues can be opened.

@irrah68
Copy link

irrah68 commented Nov 24, 2024

OK. If you can reproduce the problem I described, perhaps it is worth new issue. This pr seems to work. No problems with a blan line and braille display's navigation keys.

@nvdaes
Copy link
Collaborator Author

nvdaes commented Nov 24, 2024

@irrah68 , many thanks for testing the PR.
About selection in Notepad++, perhaps this is related to Notepad++ itself, not NVDA. I am not sure.

@seanbudd seanbudd added the triaged Has been triaged, issue is waiting for implementation. label Nov 26, 2024
@github-actions github-actions bot added this to the 2025.1 milestone Nov 27, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
app/scintilla Scintilla edit controls bug component/braille p3 https://github.com/nvaccess/nvda/blob/master/projectDocs/issues/triage.md#priority triaged Has been triaged, issue is waiting for implementation.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants