Frontmatter
| id | 3880 |
| title | "One thing i noticed when creating the PoC:" |
| state | Closed |
| labels | bug |
| assignees | [] |
| createdAt | Jan 16, 2023, 9:14 PM |
| updatedAt | Jan 24, 2023, 7:22 PM |
| githubUrl | https://github.com/neomjs/neo/issues/3880 |
| author | arcman7 |
| commentsCount | 2 |
| parentIssue | null |
| subIssues | [] |
| subIssuesCompleted | 0 |
| subIssuesTotal | 0 |
| blockedBy | [] |
| blocking | [] |
| closedAt | Jan 24, 2023, 7:22 PM |
"One thing i noticed when creating the PoC:"

tobiu
Jan 16, 2023, 9:28 PM
Hi @arcman7!
Could you please provide more input, about what you would like to change? I am assuming you mean: https://github.com/neomjs/neo/blob/dev/src/worker/Base.mjs#L137
The current version does work in FF.

arcman7
Jan 17, 2023, 1:06 AM
I'm not saying your code needs to be fixed since it works, but I just thought you should know that the "bug" you referenced in your medium article https://medium.com/swlh/chrome-v83-enables-js-module-support-for-sharedworkers-starting-a-new-era-for-multi-browser-dbb20366bddf
is actually a feature and working as intended. In your case all you had to do was call
worker.port.start()
After you used
worker.port.addEventListener('message', me.onMessage.bind(me), false);
arcman7 closed this issue on Jan 24, 2023, 7:22 PM
You mentioned in your medium article:
** "One thing i noticed when creating the PoC:**
does not work, while you can use addEventListener() on a non shared worker. Might be worth a ticket. This change solved it:
From Mozilla: