iocave / monkey-patch

Inject custom javascript into vscode
MIT License
154 stars 17 forks source link

As of September 2022 update, Customize UI changes undone on VS Code restart #51

Open emilyst opened 2 years ago

emilyst commented 2 years ago

I am still working on confirming that this is a problem with Customize UI or what might have changed, but from several tests, it seems as if restarting VS Code undoes any changes from Customize UI. (Is this a Monkey Patch problem?)

From "Help: About":

Version: 1.72.0 Commit: 64bbfbf67ada9953918d72e1df2f4d8e537d340e Date: 2022-10-04T23:21:58.256Z (1 day ago) Electron: 19.0.17 Chromium: 102.0.5005.167 Node.js: 16.14.2 V8: 10.2.154.15-electron.0 OS: Darwin x64 21.6.0 Sandboxed: No

Customize UI version v0.1.65. Monkey Patch version v0.1.21.

If anyone can reproduce this, please reply? Thanks.

emilyst commented 2 years ago

Through more testing, I have confirmed this issue does not occur on VS Code version 1.71.2. Upgrading again causes the issue again. (No extensions were changed during this test.)

I have been reading through the September 2022 milestone but cannot yet find a commit which would cause this new behavior.

ClaudiusSeibt commented 2 years ago

Same here! My Title bar styling broke as soon as I opened VS Code. I downgraded both Customize UI and Monkey Patch to privious versions, but this did not work. So I'm assuming VS Code 1.72.0 broke either Monkey Patch or Customize UI.

From "Help: About":

Version: 1.72.0 (Universal) Commit: 64bbfbf67ada9953918d72e1df2f4d8e537d340e Date: 2022-10-04T23:21:58.256Z Electron: 19.0.17 Chromium: 102.0.5005.167 Node.js: 16.14.2 V8: 10.2.154.15-electron.0 OS: Darwin arm64 21.6.0 Sandboxed: No

Customize UI version 0.1.65 Monkey Patch version 0.1.21

GJRobert commented 2 years ago

Thanks to previous reporters for opening this issue.

Here are my current workaround for reference: Found that UI customizations losing effects? Fine, just touch a little in your

    "customizeUI.stylesheet": {
    ...
    },

settings (if you don't have any yet, just write some dumb CSS there), and save settings.json, and reload the window as prompted. Voilà, UI customizations are there again.

But there is one thing I cannot make working: inline titlebar of macOS. Hopefully this extension can be fixed soon!

knopp commented 2 years ago

Can you retry with monkey-patch 0.1.22?

gee-forr commented 2 years ago

Can you retry with monkey-patch 0.1.22?

This fixes it!!! Thank you so much. VSCode is so fugly without CustomizeUI. ❤️

hk21702 commented 2 years ago

I'm still having issues even with monkey-patch updated to 0.1.22. I've tried stuff like changing the stylesheet slightly, as well as enabling and disabling monkey-patch. Customize UI is on 0.1.65

Any help would be appreciated!

Help About:

Version: 1.72.0 (user setup) Commit: 64bbfbf67ada9953918d72e1df2f4d8e537d340e Date: 2022-10-04T23:20:39.912Z Electron: 19.0.17 Chromium: 102.0.5005.167 Node.js: 16.14.2 V8: 10.2.154.15-electron.0 OS: Windows_NT x64 10.0.22621 Sandboxed: No

knopp commented 2 years ago

@hk21702, I think the problem in your case might be that you're running vscode on Windows. There might be a path separator issue with the patch that I didn't realize. I'll try to reproduce the problem.

emilyst commented 2 years ago

@hk21702, I think the problem in your case might be that you're running vscode on Windows. There might be a path separator issue with the patch that I didn't realize. I'll try to reproduce the problem.

Nota bene: https://nodejs.org/api/path.html#pathjoinpaths

emilyst commented 2 years ago

Confirming this newest version of Monkey Patch fixes the issue for me. Thanks a lot for that.

The cause is probably due to one of these numerous changes to make instantiation of the editor more "lazy." I wonder if Microsoft should implement certain kinds of "hooks" for extensions to use at startup time, assuming they haven't already.

mfreeman-xtivia commented 2 years ago

Still broken for me on Windows with version 0.1.22 of monkeypatch installed

sean-codes commented 2 years ago

On windows I did a fresh install of VS Code with only Customize UI and it did not work with latest monkeypatch

image

image

GJRobert commented 2 years ago

Monkey Patch v0.1.22 fixes this issue. Great to see that inline titlebar again!!! Thank you so much @knopp !

gencer commented 2 years ago

I really don't like to do +1 comments however, I'm pretty sure -including me- Windows is not fixed. All "fixes" mentioned here seems to be either OSX or Linux.


UPDATE:

v0.1.23 solved issues on Windows. Thanks :) 🥳

mfreeman-xtivia commented 2 years ago

v0.1.23 just fixed everything for me on Windows..Thank you !!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!

sean-codes commented 2 years ago

Thank you. v0.1.23 on Windows 11 it is working!

Haruno19 commented 1 year ago

CustomizeUI doesn't work anymore for me with MokeyPatch v0.1.23 and VS Code version 1.72.2 (on both MacOS and Windows 11). Don't know if it's still a Monkey Patch problem?

Version: 1.72.2 (Universal) Commit: d045a5eda657f4d7b676dedbfa7aab8207f8a075 Date: 2022-10-12T22:16:30.254Z Electron: 19.0.17 Chromium: 102.0.5005.167 Node.js: 16.14.2 V8: 10.2.154.15-electron.0 OS: Darwin arm64 21.6.0 Sandboxed: No

Everything worked fine a few VS Code updates ago

GJRobert commented 1 year ago

CustomizeUI doesn't work anymore for me with MokeyPatch v0.1.23 and VS Code version 1.72.2 (on both MacOS and Windows 11). Don't know if it's still a Monkey Patch problem?

Hi, I'm on VS Code v1.72.2 + Monkey Patch v0.1.23 here, on macOS 11.6, and my CustomizeUI is working fine, with all UI modifications applied, including inline titlebar. Perhaps you have to find a way re-invoke the notification of "Monkey Patch configuration changed" to get correct result again to click that "Re-apply" button on it.

Haruno19 commented 1 year ago

That's interesting... I'm on macOS 12.6, I tried changing some settings, like "customizeUI.titleBar" for example, saved the settings.json file but the notification wasn't invoked. I ran the command "Enable Monkey Patch" and it didn't return any error but didn't do anything. I tired disabling and re-enabling both the Monkey Patch and CustomizeUI extensions but that didn't work either, I tried uninstalling and reinstalling them but got the same result. I also installed VScodium instead, installed the extensions and used the same settings.json file but I got the same behaviour: no errors but the changes aren't applied.

If there's anything I can do to help, like attaching some screenshots or settings, let me know

knopp commented 1 year ago

If you don't get prompt after changing customizeUI properties it would mean the extension is not loaded properly for some reason. As for VSCodium, the extension is not tested with it so I don't know if it works. I tested both latest insiders and stable version and both seem to work for me.

Haruno19 commented 1 year ago

Ok got it. I tried to uninstall and re-install the extension again but it didn't event ask for a reload. Maybe I am missing some VS Code settings? I have no idea, it worked just fine a few days ago Can I see VS Code logs somehow?

Here's a video of its current behaviour

GJRobert commented 1 year ago

Ok got it. I tried to uninstall and re-install the extension again but it didn't event ask for a reload. Maybe I am missing some VS Code settings? I have no idea, it worked just fine a few days ago Can I see VS Code logs somehow?

Here's a video of its current behaviour

I cannot guarantee that I have found the solution, but maybe you can try making some changes in the custom CSS setting part of CustomizeUI and save the settings.json. See if this works for you.

Also, I encounter the notification of re-apply Monkey Patch configuration every time I open Code, and I can re-apply with success every time. Maybe a "install other versions" of Monkey Patch and install back to the newest version may help too.

Haruno19 commented 1 year ago

Thanks for the help! Unfortunately neither changing the CSS nor installing previous versions for the extensions (and updating it back also) worked. I'll try downgrade VS Code to an older version to see if that's the problem, otherwise I'll just wait for future updates (both for Monkey Patch and VS Code) that may fix my issue

Update:

So, I installed VS Code version 1.70 from July 2022 (when I'm sure the extensions worked fine for me), downgraded both Customize UI and Monkey Patch in order to make them work with that version of VS Code, but even this time nothing happened, the changes weren't applied and the notification wasn't invoked.

At this point I think it's clear that the problem resides only in my current setup/environment, I don't know what is that causes this behaviour but I'll try to figure it out!

Thanks everyone for your support & help :)

ArturoDent commented 1 year ago

See Engineering: private and protected properties are being mangled now.

Could be the cause of this extension's problems lately.