ALTaleX531 / TranslucentFlyouts

Translucent effect for most of the win32 flyouts
GNU Lesser General Public License v3.0
779 stars 24 forks source link

TF hates PowerToys, or vice versa #112

Closed MuscularPuky closed 3 months ago

MuscularPuky commented 4 months ago

image

Unfortunetely there's 23 splited zips. plz welcome them kindly, and remove .text suffix 23 times PowerToys.PowerLauncher.exe-minidump-2024-03-11-17-44-34.zip.txt PowerToys.PowerLauncher.exe-minidump-2024-03-11-17-44-34.z06.txt PowerToys.PowerLauncher.exe-minidump-2024-03-11-17-44-34.z05.txt PowerToys.PowerLauncher.exe-minidump-2024-03-11-17-44-34.z04.txt PowerToys.PowerLauncher.exe-minidump-2024-03-11-17-44-34.z03.txt PowerToys.PowerLauncher.exe-minidump-2024-03-11-17-44-34.z02.txt PowerToys.PowerAccent.exe-minidump-2024-03-11-17-45-03.z01.txt PowerToys.PowerAccent.exe-minidump-2024-03-11-17-45-03.zip.txt PowerToys.PowerAccent.exe-minidump-2024-03-11-17-45-03.z04.txt PowerToys.PowerAccent.exe-minidump-2024-03-11-17-45-03.z03.txt PowerToys.PowerAccent.exe-minidump-2024-03-11-17-45-03.z02.txt PowerToys.PowerLauncher.exe-minidump-2024-03-11-17-45-33.z01.txt PowerToys.PowerLauncher.exe-minidump-2024-03-11-17-45-33.zip.txt PowerToys.PowerLauncher.exe-minidump-2024-03-11-17-45-33.z09.txt PowerToys.PowerLauncher.exe-minidump-2024-03-11-17-45-33.z08.txt PowerToys.PowerLauncher.exe-minidump-2024-03-11-17-45-33.z07.txt PowerToys.PowerLauncher.exe-minidump-2024-03-11-17-45-33.z06.txt PowerToys.PowerLauncher.exe-minidump-2024-03-11-17-45-33.z05.txt PowerToys.PowerLauncher.exe-minidump-2024-03-11-17-45-33.z04.txt PowerToys.PowerLauncher.exe-minidump-2024-03-11-17-45-33.z03.txt PowerToys.PowerLauncher.exe-minidump-2024-03-11-17-45-33.z02.txt

PowerToys.PowerLauncher.exe-minidump-2024-03-11-17-44-34.z07.txt

PowerToys.PowerLauncher.exe-minidump-2024-03-11-17-44-34.z01.txt

ALTaleX531 commented 4 months ago

image Thanks for the feedback, you can temporarily add the PowerToys component to the exclusion list manually to prevent crashes, as shown in the image. I'll check out what went wrong later.

TechySC23 commented 4 months ago

I was just about to comment

TechySC23 commented 4 months ago

What's Power Launcher? I use PowerToys and there's not even a single thing! (Except power accent). And even I use PowerToys but it doesn't happen to me! What version of TF are you running @MuscularPuky?

TechySC23 commented 4 months ago

Seems that PT (PowerToys) conflicts with it @ALTaleX531

ALTaleX531 commented 4 months ago

TranslucentFlyouts.Win32.V3.x64.zip Please check if this build solves the issue.

TechySC23 commented 4 months ago

@ALTaleX531 Look at this, I was just tinkering with the GUI with some normal settings and suddenly this showed up: image

TechySC23 commented 4 months ago

The GUI suddenly closed too. It's the same error as the OP. Also, I found that Power Launcher.exe is Power Toys Run. Fortunately in my case there's only one DMP file: TranslucentFlyoutsConfig.exe-minidump-2024-03-11-21-03-18.dmp

ALTaleX531 commented 4 months ago

@TechySC23 Can you also upload your dump file? I've gotten some crash clues, but I need further confirmation.

ALTaleX531 commented 4 months ago

@TechySC23 I've reproduced the crash myself, it seems to be TFC's own problem, TF just helped catch the exception. You can ask @Satanarious what went wrong, I'm not familiar with QT. image

Satanarious commented 4 months ago

How do you reproduce this error?

ALTaleX531 commented 4 months ago

@Satanarious Here are the steps I took to reproduce the error

  1. Switch to the "Global" tab.
  2. Change the "Effect Type".
  3. Click the "Apply" button.
  4. Click the "Apply" button again.
  5. Click the "x" button
Satanarious commented 4 months ago

This error is on TF v3.0, right? Because I have a previous version and it works fine.

Edit: The apply button alters the registry, so irrespective of the TF version, it should do the job. Unless some logic in TF changed. Also, you can only click the TFC's apply button twice if you press 'Ok' on the prompt that says 'Changes Applied'. I'm using TF v2.1.0

ALTaleX531 commented 4 months ago

When I attach the debugger to TFC, TF is not currently running. I think this bug existed a long time ago because closing the window and crashing behave almost the same in front of the user.

Satanarious commented 4 months ago

Is it crashing on exit or is it somehow hindering the functionality? I mean does everything else work fine besides the crash when you try to close the application?

ALTaleX531 commented 4 months ago

I think it's just crash when closing the app, I didn't find anything else abnormal about it.

Satanarious commented 4 months ago

But how is Translucent Flyouts catching this error?

ALTaleX531 commented 4 months ago

TF calls the win32 api SetUnhandledExceptionFilter and passes in its own top-level exception filter callback function, which would normally be called if the exception was not caught by the application. These exceptions that are not caught by the application will cause the application to crash, and this function is specifically designed to detect abnormal crashes.

Satanarious commented 4 months ago

That explains it. I'll try to fix it. Please make an issue on my repo.

TechySC23 commented 4 months ago

@Satanarious and @ALTaleX531 How is your progress going for the issue? I am not even touching anything in TFC after that problem. And yes ALTaleX531 the steps to reproduce are somewhat correct. I remember changing the effect type and all the animation effects using TFC. Sadly the dump file is a whopping 323 MB so I can't upload it. I even tried 7Zip, the planet's best compressing software 🤣, choosing the Ultra compression and maxing out (Not really because sadly my laptop only has about 8 GB), it only came to about 52 MB which is an insane compression, but the max file size for GitHub is 25MB so I really CAN'T 😔.

TechySC23 commented 4 months ago

The issue is NOT "TFC will crash if you close the window after changing and applying settings in the Global tab." It is: TFC will throw an error after the user changes some effects and settings. Why not globally? In my case I changed not the global tab but the Menu and the Tooltip tab.

ALTaleX531 commented 4 months ago

@TechySC23 You can upload the dump file to Google Drive, but the TFC issue is not that important. I'm now most curious about the steps to reproduce the PowerToys crash, can you tell me how to reproduce it?

TechySC23 commented 4 months ago

Wait, PT crash? no PT didn't crash while this threw at me (error).

ALTaleX531 commented 4 months ago

@TechySC23 Oh well never mind, I thought the PowerToys crash also happened to you. So can you upload your TFC dump file?

ALTaleX531 commented 4 months ago

以及您在这个问题上的进展如何?在这个问题之后,我什至没有碰过 TFC 中的任何东西。是的,ALTaleX531,重现的步骤在某种程度上是正确的。我记得使用 TFC 更改了效果类型和所有动画效果。可悲的是,转储文件高达 323 MB,所以我无法上传它。我什至尝试了 7Zip,地球上最好的压缩软件🤣,选择超压缩并最大化(不是真的,因为可悲的是我的笔记本电脑只有大约 8 GB),它只有大约 52 MB,这是一个疯狂的压缩,但 GitHub 的最大文件大小是 25MB,所以我真的不能😔。 @Satanarious and @ALTaleX531 How is your progress going for the issue? I am not even touching anything in TFC after that problem. And yes ALTaleX531 the steps to reproduce are somewhat correct. I remember changing the effect type and all the animation effects using TFC. Sadly the dump file is a whopping 323 MB so I can't upload it. I even tried 7Zip, the planet's best compressing software 🤣, choosing the Ultra compression and maxing out (Not really because sadly my laptop only has about 8 GB), it only came to about 52 MB which is an insane compression, but the max file size for GitHub is 25MB so I really CAN'T 😔.

Haven't you tried the new build below? I remembered that I had adjusted the dump settings to prevent garbage data.

TranslucentFlyouts.Win32.V3.x64.zip Please check if this build solves the issue.

TechySC23 commented 4 months ago

No I didn't try it but I did download it. I'll try to upload my dump file. But what's up with my quoted text? It's in Chinese?

ALTaleX531 commented 4 months ago

Oh please forgive stupid MS Edge for automatically translating it again when I quoted it.

TechySC23 commented 4 months ago

what's your mail id? I am gonna mail you directly

ALTaleX531 commented 4 months ago

what's your mail id? I am gonna mail you directly

2368730049@qq.com

TechySC23 commented 4 months ago

Sent 📤

MuscularPuky commented 4 months ago

@TechySC23 You can upload the dump file to Google Drive, but the TFC issue is not that important. I'm now most curious about the steps to reproduce the PowerToys crash, can you tell me how to reproduce it?

It throws exception randomly when PowerToys startsup. I tried on 0.78.0~0.78.x (canary) and it occurs on only 0.79.0+ I think? I'm trying more case but for some reason TF does not dump anymore in desktop path. While I sleep yesterday our Windows closed my unsaved projects and force updated so I lost all application logs include TF LOL. I'm gonna track this issue again and talk to you bout this soon.

ALTaleX531 commented 4 months ago

@TechySC23 I've looked at the dump file you sent and it seems to be a TFC issue as well, as QT seems to try to read a QWindows object with a null pointer.

ALTaleX531 commented 4 months ago

@MuscularPuky 0.79.0+ is the preview version and it's indeed unstable and crashes even when I'm not running TF. The reason I asked how to reproduce it is because those exceptions recorded in those dump files don't appear to be caused by TF.

MuscularPuky commented 4 months ago

@MuscularPuky 0.79.0+ is the preview version and it's indeed unstable and crashes even when I'm not running TF. The reason I asked how to reproduce it is because those exceptions recorded in those dump files don't appear to be caused by TF.

uh I think TF hates not only PowerToys but also others I couldn't send a big file with mail so you should meet split zips again sadly

Explorer.EXE-minidump-2024-03-13-00-26-56.z01.txt Explorer.EXE-minidump-2024-03-13-00-26-56.zip Explorer.EXE-minidump-2024-03-13-00-26-56.z10.txt Explorer.EXE-minidump-2024-03-13-00-26-56.z09.txt Explorer.EXE-minidump-2024-03-13-00-26-56.z08.txt Explorer.EXE-minidump-2024-03-13-00-26-56.z07.txt Explorer.EXE-minidump-2024-03-13-00-26-56.z06.txt Explorer.EXE-minidump-2024-03-13-00-26-56.z05.txt Explorer.EXE-minidump-2024-03-13-00-26-56.z04.txt Explorer.EXE-minidump-2024-03-13-00-26-56.z03.txt Explorer.EXE-minidump-2024-03-13-00-26-56.z02.txt DevHomeGitHubExtension.exe-minidump-2024-03-15-17-17-38.z01.txt DevHomeGitHubExtension.exe-minidump-2024-03-15-17-17-38.zip DevHomeGitHubExtension.exe-minidump-2024-03-15-17-17-38.z03.txt DevHomeGitHubExtension.exe-minidump-2024-03-15-17-17-38.z02.txt

This guy's not split Explorer.EXE-minidump-2024-03-13-16-53-37.dmp

ALTaleX531 commented 4 months ago

You should not use TranslucentFlyouts.Immersive, its code has many problems and it is not ready for use. Edit: You should consider updating TranslucentFlyouts.Win32, there already exists a newer version that fixes a lot of problems.

TechySC23 commented 4 months ago

@ALTaleX531 TFC Crashed again after installing your version (v3.0.1). Want to know what I did? Well said no and kept TF!

ALTaleX531 commented 4 months ago

@TechySC23 As far as I know the latest version of TFC has solved the crash problem and you should upgrade TFC. As I said before, TFC crashes are not caused by TF, and I don't see any reason why it's worth it for you to stay stuck at v3.0.0.

TechySC23 commented 4 months ago

@ALTaleX531 I did upgrade to that (v3.0.1). Also, I complained to Satanarious about this crash. Did he release a new version?

TechySC23 commented 4 months ago

Yep he did

ALTaleX531 commented 3 months ago

The PowerToys issue can already be solved by adding the process name to the DisabledList, so the issue is obsolete , closing it now.