Maybe we should avoid browser search completely like CodeEditor does and address Ctrl+F and related shortcuts to the WikiEditor (2010)'s own search dialog

Change 375402 had a related patch set uploaded (by Niharika29; owner: Niharika Kohli):
[mediawiki/extensions/CodeMirror@master] Implement the search add-on for CodeMirror

https://gerrit.wikimedia.org/r/375402

In T174480#3566576, @Dvorapa wrote:

Maybe we should avoid browser search completely like CodeEditor does and address Ctrl+F and related shortcuts to the WikiEditor (2010)'s own search dialog

Unfortunately, the wikieditor search dialog is not really suited for most of my needs at least. It functions as a popup, and blocks a big part of the window, it does not highlight all the found phrases and it /only/ searches in the text window. Also the way it responds (once you hit enter, it looses focus on the search but turns to the edit text, so if you hit another enter to go to the next instance, it won't - etc) is not what I most frequently need when editing large pages with lots of repetition, or when i'm comparing two pages side by side. I would not support ctrl+F to be activating the wikieditor search dialog. They seem to serve a different purpose.

Maybe try to set viewportMargin to Infinity when ctrl+F pressed?

In T174480#3576427, @Pastakhov wrote:

Maybe try to set viewportMargin to Infinity when ctrl+F pressed?

https://github.com/codemirror/CodeMirror/issues/4491#issuecomment-271615629

@Niharika yes, I understand about cost. I would prefer to find a balance between speed and ease of use.

For example render everything on idle. As a variant to set viewportMargin to Infinity when the user does not change the text for some time and return one to optimal size when the user starts editing the text for reduce rendering time.

In T174480#3578800, @Pastakhov wrote:

For example render everything on idle. As a variant to set viewportMargin to Infinity when the user does not change the text for some time and return one to optimal size when the user starts editing the text for reduce rendering time.

I was wondering about that. "render everything on idle". We can try that approach and do some performance testing to see if that's more feasible.

Do remember that 'on idle' is something other than machine performance. If you take an older machine, it won't really matter when you load something, the sheer size (of memory usage and graphics size) will likely do you under. Scroll performance especially will likely tank for those users.

It's worth a try of course, but those problems are likely hard to measure (especially since the worst performing browser platforms also have the least usable metrics facilities). Tough call...

Change 376081 had a related patch set uploaded (by Niharika29; owner: Niharika Kohli):
[mediawiki/extensions/CodeMirror@master] Render everything as long as user doesn't start typing/moving cursor

https://gerrit.wikimedia.org/r/376081

Just to mention that I'm experiencing this issue using Firefox 55.0.3 on Mac OS.

Change 376081 merged by jenkins-bot:
[mediawiki/extensions/CodeMirror@master] Render everything as long as user doesn't start typing/moving cursor

https://gerrit.wikimedia.org/r/376081

For the ticket watchers - the above patch fixes the ctrl+f issues but it might show some lag when editing large articles. If this happens, please report it here along with:

1. Link to the page/section you were editing
2. Some detail about the machine you were editing on (for example: Dell, Windows, i5 processor)

Thanks.

In T174480#3578106, @Niharika wrote:

In T174480#3576427, @Pastakhov wrote:

Maybe try to set viewportMargin to Infinity when ctrl+F pressed?

https://github.com/codemirror/CodeMirror/issues/4491#issuecomment-271615629

As a side note, I have an idea that I didn't see mentioned either here or in the linked GitHub issue – could CodeMirror render all the "invisible" area, but without syntax highlighting? As I understand, it's the highlighting that is expensive, not plain text. Syntax highlighting would be applied when the text is scrolled into view. I haven't worked with CodeMirror much, so I have no idea if this might be difficult to implement.

Change 375402 abandoned by Niharika29:
[WIP] Implement the search add-on for CodeMirror

Reason:
We chose to make viewport:Infinity to render it all. Let's see how that goes.

https://gerrit.wikimedia.org/r/375402