1. `path` for the XML file is missed. 2. Timeout should be removed on destroy and before creating a new one (line 115 `extensionModules/blinkController.js`): [EGO Review Guidelines: Timeout](https://gjs.guide/extensions/review-guidelines/review-guidelines.html#remove-main-loop-sources)

2. Tis good? ``` #scheduleNextSyncBlink() { // Calculate a random interval to next blink const interval = this.#blinkIntervalRange[0] + ((this.#blinkIntervalRange[1] - this.#blinkIntervalRange[0]) * Math.random()); return setTimeout(() => { this.#blinkAll(); + this.#randomSyncTimeout = clearTimeout(this.#randomSyncTimeout); this.#randomSyncTimeout = this.#scheduleNextSyncBlink(); }, 1000 * interval); } 1) That's what I was talking about last time. In order for eyes to have their own schema, I need a template without path so that the path can be defined at creation: [Relocatable schemas](https://docs.gtk.org/gio/class.Settings.html#relocatable-schemas) ``` const eyeSchema = extension.settings.get_child('eye').settings_schema; const eyeSchemaPathRoot = `${extension.settings.settings_schema.get_path()}eyes/`; // Create eye array const count = extension.settings.get_int('eye-count'); for (let i = 1; i <= count; i++) { const eyeID = String(i).padStart(2, '0'); const eyeName = `eye-${eyeID}`; const eyeSettings = new Gio.Settings({ settings_schema: eyeSchema, path: `${eyeSchemaPathRoot}${eyeName}/`, }); extension.eyeArray.push(new Eye(extension, eyeSettings, eyeName)); } ``` This, with line 342 `prefsModules/eyeSettings.js`, adds paths such as '/org/gnome/shell/extensions/eye-on-cursor/eyes/eye-01', '/org/gnome/shell/extensions/eye-on-cursor/eyes/eye-02', etc., for each eye object. Having a path in the xml when trying to create the schema with a new one leads to an error (GSettings created for path '/org/gnome/shell/extensions/eye-on-cursor/eyes/eye-01/', but schema specifies '/org/gnome/shell/extensions/eye-on-cursor/eye/'). And having just a static path leads back to the beginning of eyes sharing the same settings. Using `get_child` instead on `lookup` to find the schema doesn't address the problem.

For the timeout. No. That's not good. `setTimeout()` returns the timeout id, that's what you should store and remove.

I think I see? ``` #scheduleNextSyncBlink() { // Calculate a random interval to next blink const interval = this.#blinkIntervalRange[0] + ((this.#blinkIntervalRange[1] - this.#blinkIntervalRange[0]) * Math.random()); this.#randomSyncTimeout = clearTimeout(this.#randomSyncTimeout); this.#randomSyncTimeout = setTimeout(() => { this.#blinkAll(); this.#scheduleNextSyncBlink(); }, 1000 * interval); } ``` Now it mirrors `#scheduleNextBlink(eye)`. Do I do the same there?

That's correct now. Don't forget to also remove that timeout source id on destroy.