Skip to content

feat: restore icon and context menu when Explorer restarts if necessary #71

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

Merged

Conversation

pzhlkj6612
Copy link
Contributor

Hello! I'm going to fix #65.

To maintainers

To preserve the hard work from @brandon3343, I've being based on #39. Please don't forget to squash all commits into one before merging.

Test

The screen recordings for comparison:

Original version (973525c) Fixed version
leanflutter-tray_manager-windows-taskbar-awareness-original.mp4
leanflutter-tray_manager-windows-taskbar-awareness-fixed.mp4

My test steps:

  1. Open our example app and click "setContextMenu".
  2. Click "popUpContextMenu" and click an item.
  3. Set an icon and click it to show the menu, then click an item.
  4. Click "popUpContextMenu" and click an item.
  5. Restart Windows Explorer (e.g., by running powershell -command "gps explorer | spps"), then check system tray.
  6. Click "popUpContextMenu" and click an item.
  7. Click the icon to show the menu, then click an item.
  8. Set an icon and click it to show the menu, then click an item.
  9. Click "popUpContextMenu" and click an item.
  10. Destroy the icon and set an icon, then click the icon to show the menu and click an item.
  11. Click "popUpContextMenu" and click an item.
  12. Destroy the icon.
  13. Click "popUpContextMenu" and click an item.
  14. Restart Windows Explorer, then check system tray.
  15. Click "popUpContextMenu" and click an item.

The environment:

  • OS: Windows 10 (10.0.15063); also tested on Windows 10 (10.0.19045.4780) and Windows 11 (10.0.22631.4317)
  • Recorder: FastStone Capture 11.0

.

@pzhlkj6612 pzhlkj6612 changed the title feat: restore icon and context menu when Explorer restarts if possible feat: restore icon and context menu when Explorer restarts if necessary Apr 24, 2025
@pzhlkj6612
Copy link
Contributor Author

Sorry for being confusing. The title of this PR and the message of the key commit has been corrected:

- feat: restore icon and context menu when Explorer restarts if possible
+ feat: restore icon and context menu when Explorer restarts if necessary

We should NOT take care of possibility, but necessity.

@lijy91
Copy link
Member

lijy91 commented Apr 25, 2025

LGTM

@lijy91 lijy91 merged commit 0c655a1 into leanflutter:main Apr 25, 2025
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Restart Windows Explorer in Task Manager, and the previously set APP icon will disappear.
3 participants