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

[BUG] Count (or FindNext) works, but FindAllinCurrDoc doesn't #15779

Open
1 task done
alankilborn opened this issue Nov 10, 2024 · 1 comment
Open
1 task done

[BUG] Count (or FindNext) works, but FindAllinCurrDoc doesn't #15779

alankilborn opened this issue Nov 10, 2024 · 1 comment

Comments

@alankilborn
Copy link
Contributor

Is there an existing issue for this?

  • I have searched the existing issues

Description of the Issue

Please see STR (Steps to Reproduce), below.

Steps To Reproduce

  1. Create a new tab and add this text:
abc\r\n
123

def\r\n
456

abc\r\n
123

def\r\n
456
  1. Select lines 1 through 2 fully by clicking on the 1 in the line number margin and dragging downward to the 2; result:
    image
  2. Press Ctrl+f to invoke the Find window (Find what gets auto-populated with selected text)
  3. Setup the other find options as shown here:
    image
  4. Press Count; observe 2 occurrences counted (good)
  5. Without changing anything else, press Find All in Current Document

Current Behavior

Zero occurrences found:

image

Expected Behavior

2 occurrences (obviously)

Debug Information

Notepad++ v8.7   (64-bit)
Build time : Sep 17 2024 - 17:06:31
Path : C:\NotepadPlusPlus\Misc\_NoBackup\Releases\npp.8.7\npp.8.7.portable.x64\notepad++.exe
Command Line : -titleAdd="8.7 64-bit" -multiInst
Admin mode : OFF
Local Conf mode : ON
Cloud Config : OFF
Periodic Backup : ON
OS Name : Windows 11 Pro (64-bit)
OS Version : 23H2
OS Build : 22631.4317
Current ANSI codepage : 1252
Plugins : 
    mimeTools (3.1)
    NppConverter (4.6)
    NppExport (0.4)
    PythonScript (2)

Anything else?

The STR were shown using the Count function, but correct searching behavior can be shown with Find Next instead by substituting this new step 5 for the step 5 in STR:

  1. Press Find Next repeatedly; observe that Notepad++ will find the data at lines 7 through 8, then 1 through 2, then 7 through 8, etc. (good)
@softmgr
Copy link

softmgr commented Nov 11, 2024

It looks like the escape characters were not handled correctly.

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

2 participants