electron-findbar 0.2.0 → 0.3.0

package/README.md

@@ -82,12 +82,20 @@ findbar.setWindowHandler(win => {
 
 ### Opening the Findbar
 
-The Findbar is a child window of the `BaseWindow` passed during construction. It will follow the relative movement of the parent window and must be visible for the Findbar to display:
+The Findbar is a child window of the `BaseWindow` passed during construction. To open it use:
 
 ```js
 findbar.open();
 ```
 
+On MacOS, the findbar will follow the relative movement of the parent window by default and there is no way to change it. On Windows and Linux, this behavior is not default and is simulated by using the `move` event of the parent and can be disabled by using:
+
+```js
+findbar.followParentWindow(false)
+```
+
+> Enabled by default.
+
 ### Finding Text in the Page
 
 Once open, the Findbar appears by default in the top-right corner of the parent window and can be used without additional coding. Alternatively, you can use the following methods to trigger `findInPage` and navigate through matches in the main process:

package/index.js

@@ -23,6 +23,12 @@ class Findbar {
     /** @type {string} */
     #lastValue = ''
 
+    /**
+     * Workaround to fix "findInPage" bug - double-click to loop
+     * @type {boolean | null}
+     */
+    #fixMove = null
+
     /**
      * Prepare the findbar.
      * @param {BaseWindow} parent Parent window.
@@ -54,7 +60,7 @@ class Findbar {
 
         this.#windowHandler && this.#windowHandler(this.#window)
         
-        this.#window.loadFile(path.join('web', 'findbar.html'))
+        this.#window.loadFile(path.join(__dirname, 'web', 'findbar.html'))
     }
 
     /**
@@ -87,6 +93,10 @@ class Findbar {
      * Select previous match if any.
      */
     findPrevious() {
+        if (this.#matches.active === 1) {
+            this.#fixMove = false
+        }
+
         this.#searchableContents.findInPage(this.#lastValue, { forward: false })
     }
 
@@ -94,6 +104,10 @@ class Findbar {
      * Select next match if any.
      */
     findNext() {
+        if (this.#matches.active === this.#matches.total) {
+            this.#fixMove = true
+        }
+
         this.#searchableContents.findInPage(this.#lastValue, { forward: true })
     }
 
@@ -183,10 +197,7 @@ class Findbar {
         })
 
         this.#searchableContents.prependOnceListener('destroyed', () => { this.close() })
-        this.#searchableContents.prependListener('found-in-page', (_e, result) => {
-            this.#matches = result
-            this.#sendMatchesCount(result.activeMatchOrdinal, result.matches)
-        })
+        this.#searchableContents.prependListener('found-in-page', (_e, result) => { this.#sendMatchesCount(result.activeMatchOrdinal, result.matches) })
     }
 
     /**
@@ -207,7 +218,14 @@ class Findbar {
      * @param {number} total Total matches.
      */
     #sendMatchesCount(active, total) {
-        this.#window.webContents.send('electron-findbar/matches', { active, total })
+        if (this.#fixMove !== null) {
+            this.#fixMove ? this.findNext() : this.findPrevious()
+            this.#fixMove = null
+        }
+
+        this.#matches = { active, total }
+
+        this.#window.webContents.send('electron-findbar/matches', this.#matches)
     }
 
     /**

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "electron-findbar",
-  "version": "0.2.0",
+  "version": "0.3.0",
   "description": "Chrome-like findbar for your Electron app.",
   "main": "index.js",
   "scripts": {