anomalyco/opencode
1
0
Fork 0
mirror of https://github.com/anomalyco/opencode.git synced 2026-07-21 17:56:33 -04:00
Code Issues 5.9k Packages Projects Releases 19 Wiki Activity

[PR #9369] fix(desktop): apply getComputedStyle polyfill on all platforms #13090

New Issue
Closed
opened 2026-02-16 18:17:58 -05:00 by yindo · 0 comments
yindo commented 2026-02-16 18:17:58 -05:00
Owner
Copy Link
Copy Source

Original Pull Request: https://github.com/anomalyco/opencode/pull/9369

State: closed
Merged: Yes

Floating UI can call getComputedStyle with non-elements (e.g., null refs, virtual elements) on all platforms, not just Windows. This removes the Windows-only condition so the safety wrapper works on macOS and Linux too.

Fixes TypeError: Argument 1 ('element') to Window.getComputedStyle must be an instance of Element

What does this PR do?

On MacOS, I got error with the following:

TypeError: Argument 1 ('element') to Window.getComputedStyle must be an instance of Element
getComputedStyle@[native code]
ku@tauri://localhost/assets/index-D9T6R7qf.js:1707:89841
ru@tauri://localhost/assets/index-D9T6R7qf.js:1707:91735
wN@tauri://localhost/assets/index-D9T6R7qf.js:1707:99254
@tauri://localhost/assets/index-D9T6R7qf.js:1707:105255
Cb@tauri://localhost/assets/index-D9T6R7qf.js:2:8252
$l@tauri://localhost/assets/index-D9T6R7qf.js:2:7983
Ld@tauri://localhost/assets/index-D9T6R7qf.js:2:9468
Qk@tauri://localhost/assets/index-D9T6R7qf.js:2:10732
yn@tauri://localhost/assets/index-D9T6R7qf.js:2:9656
Jk@tauri://localhost/assets/index-D9T6R7qf.js:2:10317
yn@tauri://localhost/assets/index-D9T6R7qf.js:2:9668
M@tauri://localhost/assets/index-D9T6R7qf.js:2:3736

Removes the if (isWindows) condition from the getComputedStyle polyfill in packages/desktop/src/index.tsx

Applies the safety wrapper on all platforms (macOS, Windows, Linux) instead of Windows only

How did you verify your code works?

Tested on local test cases. I can't reproduce this error on desktop version. It pop out sometimes when I continue on a conversation. But I am sure that this error occurs on MacOS.

**Original Pull Request:** https://github.com/anomalyco/opencode/pull/9369 **State:** closed **Merged:** Yes --- Floating UI can call getComputedStyle with non-elements (e.g., null refs, virtual elements) on all platforms, not just Windows. This removes the Windows-only condition so the safety wrapper works on macOS and Linux too. Fixes TypeError: Argument 1 ('element') to Window.getComputedStyle must be an instance of Element ### What does this PR do? On MacOS, I got error with the following: ``` TypeError: Argument 1 ('element') to Window.getComputedStyle must be an instance of Element getComputedStyle@[native code] ku@tauri://localhost/assets/index-D9T6R7qf.js:1707:89841 ru@tauri://localhost/assets/index-D9T6R7qf.js:1707:91735 wN@tauri://localhost/assets/index-D9T6R7qf.js:1707:99254 @tauri://localhost/assets/index-D9T6R7qf.js:1707:105255 Cb@tauri://localhost/assets/index-D9T6R7qf.js:2:8252 $l@tauri://localhost/assets/index-D9T6R7qf.js:2:7983 Ld@tauri://localhost/assets/index-D9T6R7qf.js:2:9468 Qk@tauri://localhost/assets/index-D9T6R7qf.js:2:10732 yn@tauri://localhost/assets/index-D9T6R7qf.js:2:9656 Jk@tauri://localhost/assets/index-D9T6R7qf.js:2:10317 yn@tauri://localhost/assets/index-D9T6R7qf.js:2:9668 M@tauri://localhost/assets/index-D9T6R7qf.js:2:3736 ``` Removes the if (isWindows) condition from the getComputedStyle polyfill in packages/desktop/src/index.tsx Applies the safety wrapper on all platforms (macOS, Windows, Linux) instead of Windows only ### How did you verify your code works? Tested on local test cases. I can't reproduce this error on desktop version. It pop out sometimes when I continue on a conversation. But I am sure that this error occurs on MacOS.
yindo added the pull-request label 2026-02-16 18:17:58 -05:00
yindo closed this issue 2026-02-16 18:17:58 -05:00
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
Clear labels
acp
bad
bug
discussion
docs
documentation
enhancement
extension
good first issue
growth
help-wanted
mcp
model-problem
needs:compliance
nix
opentui
perf
pull-request
Mirrored from GitHub Pull Request
 tui
web
windows
zen
No Label pull-request
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
yindo
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: anomalyco/opencode#13090
Delete Branch "%!s()"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?