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

Select and Find Next: Create the Find Dialog Invisibly #13381

Closed
Yaron10 opened this issue Mar 20, 2023 · 9 comments
Closed

Select and Find Next: Create the Find Dialog Invisibly #13381

Yaron10 opened this issue Mar 20, 2023 · 9 comments

Comments

@Yaron10
Copy link

Yaron10 commented Mar 20, 2023

STR:
Open NPP.
Do not open the Find dialog.
Select and Find Next.

Result:
The Find dialog is displayed for a split second and then hidden.
This causes a "flickering effect".

Expected:
Avoid the flickering and create the Find dialog invisibly.


Notepad++ 8.5.1.

@alankilborn
Copy link
Contributor

#9536 ?

@Yaron10
Copy link
Author

Yaron10 commented Mar 20, 2023

@alankilborn,

👍
Does the "Walking Encyclopedia" remember ALL issues? :)
Let's keep this one open as I've just referenced it in #13372 (comment).

@alankilborn
Copy link
Contributor

Does the "Walking Encyclopedia" remember ALL issues?

No, but I remembered this from our fairly recent discussion of the flicker when doing the recent "volatile" changes.

@Yaron10
Copy link
Author

Yaron10 commented Mar 20, 2023

Well, I don't even remember the flicker being mentioned in the "volatile" discussion.

@alankilborn
Copy link
Contributor

alankilborn commented Mar 20, 2023

I can't currently find it, but I know it came up along with "volatile"...
Regardless, this StR shows it:

Open NPP.
Do not open the Find dialog.
Find (Volatile) Next.

@Yaron10
Copy link
Author

Yaron10 commented Mar 20, 2023

You probably mean that if your "volatile" proposal had been implemented, we would have had the flicker there as well.
Currently, Find (Volatile) Next does not create the Find dialog.

@alankilborn
Copy link
Contributor

alankilborn commented Mar 20, 2023

You probably mean that if your "volatile" proposal had been implemented, we would have had the flicker there as well.
Currently, Find (Volatile) Next does not create the Find dialog.

Ah yes, you are correct; ignore my earlier babble in my previous posting. 😥

@Yaron10
Copy link
Author

Yaron10 commented Mar 20, 2023

I suppose you've implemented your proposal for your personal use. :)
This happened to me here. I didn't even remember I changed that.

@Yaron10
Copy link
Author

Yaron10 commented Mar 23, 2023

Related:
#13396.

@donho donho closed this as completed in 1f96776 Mar 24, 2023
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

Successfully merging a pull request may close this issue.

2 participants