notepad-plus-plus / npp-usermanual

Notepad++ User Manual
https://npp-user-manual.org/
Other
215 stars 98 forks source link

Possible general Wrap-around section improvement #651

Closed alankilborn closed 5 months ago

alankilborn commented 5 months ago

There appears to be kind of an overview/general statement about Wrap around here:

https://npp-user-manual.org/docs/searching/#:~:text=If%20checked%2C%20when%20the%20search%20reaches%20the%20end%20of%20the%20document%2C%20it%20will%20wrap%20around%20to%20the%20beginning%20and%20continue%20searching.

This is "okay", but here's my experience: I was wanting to find a manual reference to cite because of https://github.com/notepad-plus-plus/notepad-plus-plus/issues/14909#issuecomment-2022468265 . I found the first Searching section reference to Wrap around and basically stopped looking for more (thinking that there weren't any more). Further looking later led me to a more and better reference.

I'm wondering if it wouldn't be better to replace the really really simple "If checked, when the search reaches the end of the document, it will wrap around to the beginning and continue searching" with a bit more of a complete picture, e.g.:


Wrap around


But OTOH it is totally fine if the first statement about Wrap around is meant to be dead simple (as it currently is).

pryrt commented 5 months ago

Makes sense to me. Let me know if #655 meets your expectations.

alankilborn commented 5 months ago

Looks good but I did comment on 1 place in the "code". Up to your discretion (like everything) if you want to keep that the way it is or change it in some way.