-
-
Notifications
You must be signed in to change notification settings - Fork 1k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fix Start button context menu #2174
base: master
Are you sure you want to change the base?
Conversation
@Amrsatrio What's your input on this? On my side, |
Can confirm even on RP builds without symbols, ofc without using any tweaks other than TranslucentTB and OldNewExplorer. They're just broken due to software that hooks our needed functions before we find for them (when symbols are not yet available). The real thing that needs fixing is making the pattern scans scan the DLL from the filesystem instead from memory to allow compatibility with other tweak software. In my opinion, I'm just afraid that we have to keep the mantra "if it works, don't fix it". |
for me, right click on start menu doesn't work at all. i could check if build with proposed fix works, if someone published binary of it. |
@mexmer If that happens with latest EP and latest version of 22000 please make an issue. Afaik the latest one is 22000.2416 |
i'm somewhat limited regarding updating release version of windows, in work we have central management, which keeps us at 21H2, until 22H2 is officialy approved. (mind it's enterprise edition of windows) |
but seems after restart, with lastest build of explorer patcher, right menu is back ... so i would say it works. maybe update was not complete (i did restart explorer.exe trough task manager, before) |
I looked into this again and I can say that it doesn't work for me anywhere I've tested it, and really shouldn't work. The second parameter to the It could be fixed by changing the type of that parameter, but I'd like to mention that there isn't any need to do it this way anyway. We already know the atom from registering the class! Just pass it as I did in my code. |
This is a more straightforward change I made that fixes the right click menu over the Start button, which wasn't working at all.
I was going to include a fix for the
EnsureXAML
function but I see @Amrsatrio already submitted something along those lines. In fact, I'm almost positive that whole routine can be deleted. I've deleted it in my fork.