! Rushabh | Linux Shill - hey everyone need you...

nice

showing names two time , maybe we can prefer only one :

they both have different purpose

oh they are breadcrumbs ?

yeah

the top drop down is used in small screen

so ig we can remove it for large screen

https://github.com/tscircuit/tscircuit.com/pull/3966

<@&1420812538421051555> please take a look everyone, dont spare my feelings!!!

actually this is missing something important .. the "format and import" !

yeah we are not addig it in the editor

it is not a editor specific featuer, we will move it to the website itself

where should we add the option

we can add format in the editor ig

idk maybe somewhere here ? near the download or something ?

yeah

let see

anything else

this editor is super cool actually .. no laggy ! 🔥

hey <@452450065941004289> .. check this out !

combined the search with dropdown like the vscode style ..

wait i have even better

It's amazing 🙌 🔥 <@452450065941004289>

But this feels like having a new filebar inside that ?

i followed vscode

it doesn't give me any sense that here is the search ! like the search is hidden ?

ooh ok i never used that bread crump in vscode .

vs code also allow you to search

in the breadcrump

it just does not show you a seach box

how can i search ?

in vscode if you click on the breadcrumb and start typing it searches

but it is shallow search

our implemetation also search inside the dir

can you find any other problems

have you guys tried right click

maybe we can keep the import in the center . Download | import | copy URL or you can match the style of download button ?

<@452450065941004289> please take this a look .. we have something cmd+p to search files .. we need to some how express that to users . ig this design looks clean <@&1420812538421051555> comments please : https://github.com/tscircuit/monaco-code-editor/pull/48

this crashed my browser

how ?

maybe it was out of memory panic, I don't know. when I clicked run it just froze and crashed

maybe a coincidence

working fine for me

https://github.com/tscircuit/tscircuit.com/pull/3966 <@757706909351411845> <@809856037376491570>

Preview: https://tscircuit-ffeuulirw-tscircuit.vercel.app/editor?template=blank-circuit-board

Mobile doesnt work for me, crashes

I can't seem to reproduce on my device.

<@&1420812538421051555> can you guys try in your devices

maybe browser?

i think its a safari specific bug then

+1

working fine for me in safari desktop

Yea idk hard to debug

bad :/

Iphone 15

Can you try changing browser so that we can identify if it is a browser issuse or somethig else.

<@&1420812538421051555> with an i phone please help!!

It would be nice if we get this resolved asap, i wanted to get this merged before open sauced, i believe our site will get more traffic for few days.

Crashes for me too, both safari and chrome

at my cpu for only a minute. I do really like it, is it possible to remove these autocomplete options

i'd like to just see tscircuit stuff, i think the other stuff generally shouldn't be used anyway

probably not possible- but would be nice- maybe we can prioritize properties or something?

not a huge deal- i think the crash is really the only thing blocking merge

oh also the mobile with the sidebar is pretty bad/hard to use- it only loads for a second for me but definitely shouldn't be showing me a sidebar

<@452450065941004289> it might be possible that this is a general issue w/ monaco and mobile or something due to types being loaded etc.

I think it is a ios specific bug, anyone with an iphone who would like to help me resolve this.

FWIW a crash I had was desktop chrome. Not sure if it's the same issue. Only reproduces as I hit run. Maybe something related to workers? Some memory corruption, maybe 0-day bug, browsers shouldn't crash like that consistently. Unless it's a memory leak, which tracks as web workers have limited resources budget. But taking the whole browser down with it?

I have 8Gib of RAM, btw

``` getWorker(_workerId: string, label: string) { if (label === "typescript" || label === "javascript") { return new tsWorker() } return new editorWorker() }, ``` I guess it should terminate workers on some event or use singleton or cache by workerId? I guess workerId parameter exists for a reason?

Could it be some ut with mount/unmount and dispose not being called, so workers are not disposed of? just speculating.