HealKnix / macos-titlebar-for-windows

https://marketplace.visualstudio.com/items?itemName=healknix.macos-titlebar-for-windows
2 stars 0 forks source link

Crash on latest vscode 1.93.1 #5

Open 4fort opened 2 months ago

4fort commented 2 months ago
[error] Error: Titlebar overlay is not enabled
    at CodeWindow.updateWindowControls (/usr/share/code/resources/app/out/vs/code/electron-main/main.js:117:120297)
    at CodeWindow.n (/usr/share/code/resources/app/out/vs/code/electron-main/main.js:117:117930)
    at new C (/usr/share/code/resources/app/out/vs/code/electron-main/main.js:117:123288)
    at new CodeWindow (/usr/share/code/resources/app/out/vs/modules/patch.main.js:70:11)
    at $.o (/usr/share/code/resources/app/out/vs/code/electron-main/main.js:64:1608)
    at $.createInstance (/usr/share/code/resources/app/out/vs/code/electron-main/main.js:64:1100)
    at j.rb (/usr/share/code/resources/app/out/vs/code/electron-main/main.js:118:46308)
    at j.cb (/usr/share/code/resources/app/out/vs/code/electron-main/main.js:118:34790)
    at j.Z (/usr/share/code/resources/app/out/vs/code/electron-main/main.js:118:33699)
    at j.open (/usr/share/code/resources/app/out/vs/code/electron-main/main.js:118:29689)
    at async yt.startup (/usr/share/code/resources/app/out/vs/code/electron-main/main.js:118:58105)
    at async Fe.a (/usr/share/code/resources/app/out/vs/code/electron-main/main.js:120:1275)

this is the main.log

HealKnix commented 2 months ago

After the new VS Code update the developers may have messed with electron. That's why Apc Customize UI++ extension is broken. So we are waiting for this extension to be updated. If it is updated...

4fort commented 2 months ago

After the new VS Code update the developers may have messed with electron. That's why Apc Customize UI++ extension is broken. So we are waiting for this extension to be updated. If it is updated...

so apparently this extension left an apc stylesheet to settings.json and surprisingly still works with this extension disabled.

Also I’m on linux and disabling the experimental window controls button to match with native on custom might be the reason why it worked.

I don’t know if I should close this issue or not since what worked to me isn’t due to fixing the extension.

HealKnix commented 2 months ago

Hmm... That's interesting))) So even with APC turned off, you still see the custom titlebar? Maybe it's just a quirk of the operating system. I’m on Win 11, and I can't afford to do that.

4fort commented 2 months ago

Hmm... That's interesting))) So even with APC turned off, you still see the custom titlebar? Maybe it's just a quirk of the operating system. I’m on Win 11, and I can't afford to do that.

I don't think you can apply custom css to vscode without APC.

I think you're trying to say is APC not working on your latest vscode which it didn't to me aswell on previous version after 1.91; Try updating to 1.93.

HealKnix commented 2 months ago

We seem to have misunderstood each other a bit. I have the latest version of VS Code 1.93.1 installed. The styles related to the titlebar stopped working about three weeks ago, when the 1.93 update was released. People have already written to me about this issue, and I also noticed that the styles for the titlebar stopped working. Then I dug up some information in the Apc Customize UI++ repository, and other people also had issues after this update. As of now, APC still hasn't been updated. That's the situation :p

HealKnix commented 2 months ago

The tricky part is that the custom titlebar only works when

"apc.electron": {
        "titleBarStyle": "hiddenInset" 
}

is enabled. But on Windows, this causes VS Code to crash at startup.

HealKnix commented 2 months ago

I can't say anything about other operating systems since I don't have access to them.

HealKnix commented 2 months ago

And yeah, about the custom css. I wasn't saying it would work without Apc))) I understand that without Apc the styles won't work, which is why I was surprised that everything stayed the same for you, even with Apc turned off. 0_o

4fort commented 2 months ago

The tricky part is that the custom titlebar only works when

"apc.electron": {
        "titleBarStyle": "hiddenInset" 
}

is enabled. But on Windows, this causes VS Code to crash at startup.

Correct! About that as well, I commented that line out and my vscode works with APC enabled and without crashing. I believe that is what lead to vscode crashing with APC. saw it here

HealKnix commented 2 months ago

Oh, forget what I said. I misunderstood you. I thought you had titlebar styles working, but you just meant css styles that are not related to the titlebar xD

HealKnix commented 2 months ago

I was reading between the lines, so I apologize for my misunderstanding.

HealKnix commented 2 months ago

And so yes, the problem still remains. I hope that the creator of Apc will fix it, otherwise I will have to deprecate the extension

4fort commented 2 months ago

Oh, forget what I said. I misunderstood you. I thought you had titlebar styles working, but you just meant css styles that are not related to the titlebar xD

Lol, actually it is the titlebar image

I was reading between the lines, so I apologize for my misunderstanding.

You're good 😂

HealKnix commented 2 months ago

So... I'm definitely broken now))))) For clarification, has it always been there or has it recently appeared? I can't get the gist of it xD

HealKnix commented 2 months ago

God, it's been a long time since I've seen that traffic light...