Fix: theme toggle each time content script is loaded

[tr4cks]

* denotes required fields

Priority*

• High: This PR needs to be merged first, before other tasks.
• Medium: This PR should be merged quickly to prevent conflicts due to common changes. (default)
• Low: This PR does not affect other tasks, so it can be merged later.

Purpose of the PR*

Make sure the theme doesn't change every time the content script is loaded into a page. This can be a bit confusing the first time you test the project.

Changes*

I removed the toggleTheme() function in the content script and the dependency in package.json. I added an external function to still show an example of how to define functions in an external module.

How to check the feature

Open the extension's side panel and then search in the browser's search bar. The theme no longer changes every time you change pages (when the content script is loaded).

[github-actions]

Thank you for your contribution. We will check and reply to you as soon as possible.

[PatrykKuniczak]

@tr4cks You've forgoten to run pnpm i after you've removed '@extension/storage' from package.json let's run it and commit pnpm-lock.json file

[tr4cks]

is it normal that the update also affected another line?

From:

expect-webdriverio: 5.0.2(@wdio/globals@9.4.5(@wdio/logger@9.1.0))(@wdio/logger@9.4.4)(webdriverio@9.4.5)

To:

expect-webdriverio: 5.0.2(@wdio/globals@9.4.5(@wdio/logger@9.4.4))(@wdio/logger@9.4.4)(webdriverio@9.4.5)

Other lines have been modified, but they also concern the same thing.

Let me know if I leave it the way it was or update that too.

Thanks

[PatrykKuniczak]

@tr4cks Probably it have ^ in version inside this package, and on package change pnpm pull latests version with minor changes , that's why it change.

I'm not very sure about that, but that's probably it, but after all, run pnpm i and push --force-with-lease here :)

[PatrykKuniczak]

LGTM