Windows Explorer Integration: Added support for other languages & tabs (Win 11) + code refactoring by xavier-ateo · Pull Request #2020 · talonhub/community

xavier-ateo · 2025-09-08T14:47:19Z

The file picker GUI for Explorer did not work for me on Windows 11 24H2. I have addressed several issues:

Multiple language support
Previously, the Windows Explorer integration was only targeted towards English as the operating system language. It now also supports German, and other languages can be easily added.

Windows 11 tabs
Path detection was broken when multiple tabs were open in one Explorer window. This is now fixed for English and German.

Logging and code refactoring
Logging can be easily enabled to debug issues and figure out which format a specific language uses when you want to add support for it.

Todo

I took over the previously existing logic for determining OneDrive user folders. However I did not test if it still works as intended.
I also did not test it under Windows 10.

…orer tabs in Windows 11 + code refactoring

for more information, see https://pre-commit.ci

lunixbochs · 2025-09-08T22:53:47Z

+
+# Configure logging
+if DEBUG_LOGGING:
+    logging.basicConfig(level=logging.DEBUG, format="%(levelname)s: %(message)s")


remove this please, talon already enables logging

lunixbochs · 2025-09-08T22:55:55Z

+            ],
+            "folder_names": {  # Right side is the localized name
+                "Desktop": "Desktop",
+                "Documents": "Dokumente",


surely there's a win32 api to get the names for these folders rather than hardcoding per language

You are right, there is a win32 API. I will use that instead of the hardcoded values for the user folder names.

nriley · 2025-09-13T16:28:09Z

 from talon import Context, Module, actions, app, ui

+# Configuration
+DEBUG_LOGGING = False


In general, we tend to use Talon settings rather than constants in the code to configure logging verbosity.

nriley · 2025-09-13T16:28:23Z

+if DEBUG_LOGGING:
+    logging.basicConfig(level=logging.DEBUG, format="%(levelname)s: %(message)s")


If you want to declare a logger that is private to your module you can use something like

community/core/command_client/rpc_client/rpc_client.py

Line 13 in 14e89d6

logger = logging.getLogger(__name__)

. Then generally you can just set the log level based on the Talon setting as above and the overhead from constructing log messages will be negligible when compared with the verbosity repeated checks add to your code.

I'll do that, thanks

nriley · 2025-09-13T16:30:35Z

+        current_language_mapping["folder_names"], get_folder_path, user_display_name
+    )
+
+    directories_to_exclude = [""] + current_language_mapping["excludes"]


Can you explain why you have an empty string in this list?

It was part of the original implementation, therefore I kept it. I don't know if it is needed or not.

directories_to_exclude = [ "", "Run", "Task Switching", "Task View", "This PC", "File Explorer", "Program Manager",

nriley · 2025-09-13T16:31:23Z

 class UserActions:

    def file_manager_open_parent():
+        """Navigate to parent directory."""


In general, we do not put docstrings in a context implementation, just in the module definition of an action.

The original implementation also had docstrings. Should I remove all of them?

Please, as long as there is nothing missing from the module docstring.

… API integration Refactoring of previous Windows Explorer integration: **Improvements:** - Use Windows API for user folder display names instead of hardcoded values - Use Windows API for user folder paths. This removes the need for the previous OneDrive user folder detection - Simplified language mapping structure and removed merging logic - Reorganized code into logical sections **Code review adjustments:** - Use "logger = logging.getLogger(__name__)" for logging - Removed docstrings from UserActions methods

for more information, see https://pre-commit.ci

xavier-ateo · 2025-09-15T15:58:37Z

I have reworked the code quite a bit:

It now uses Windows APIs for resolving the user folder display names and paths. This removes the need for the previous OneDrive logic.
It now uses a local logger instance which can be enabled when adding a new langauge mapping
Removed docstrings from UserActions methods
Lots of code restructuring

What is still an issue:

Nested localized standard folders (e.g., "Pictures/Screenshots") don't work with the file manager commands
WSL root folders (e.g., "Linux/Ubuntu") don't work with the file manager commands

Languages still need to be implemented manually because:

Windows 11 returns a localized window title if multiple tabs are open ("Documents and 1 more tab - File Explorer"). I could not find a way to solve this via an API.
Window titles such as "Gallery", "Home", "Network" I haven't yet figured out. Will look into it though.

What I also did:

Removed the following excluded window titles, because I could not see when the Explorer would ever show those as titles:
"Task Switching", "Task View", "Program Manager", "Run"

nriley

From the community backlog session — apologies for the delay in review, as we did not realize that you had made some additional changes. This is looking really good! Do you want to have us consider merging it soon and make the other changes in a future PR?

nriley · 2025-10-25T16:26:57Z

+    guid.Data1 = int(guid_string[0:8], 16)
+    guid.Data2 = int(guid_string[8:12], 16)
+    guid.Data3 = int(guid_string[12:16], 16)


The Python uuid.UUID object can give you these with .time_low, .time_mid and .time_hi_version. I don't see a way to get 8 individual bytes so you'll have to do that yourself!

xavier-ateo · 2025-10-27T09:47:25Z

No worries about the delay! I don't plan to make any additional changes. You can merge it anytime you want.

nriley · 2025-12-27T17:41:34Z

Apologies for the delay in updating. Will try to get to this over the weekend.

knausj85 · 2026-01-24T18:55:41Z

@@ -92,14 +475,16 @@ def file_manager_open_parent():

    def file_manager_current_path():
        path = ui.active_window().title


For what it's worth, I switched my implementation of this function to use the below, as it provides the actual path. It also resolves #1386 without using accessibility

def get_active_explorer_path(): shell = win32com.client.Dispatch("Shell.Application") active_window = ui.active_window() for window in shell.Windows(): try: if window.HWND == active_window.id: path = window.Document.Folder.Self.Path print(f"{path}") return path except Exception: continue return None

I wonder if (a) it works properly for other languages and (b) might be more robust and simplify the overall logic here. For e.g. the Desktop folder, it provides the full path. For "special" folders like "This PC," it returns a UUID.

The caveat is that I currently use the dynamic list implementation, so I don't know how well this would play with the title change approach used by the current implementation. There could be timing issues.

xavier-ateo and others added 2 commits September 8, 2025 16:29

Windows Explorer Integration: Added support for other languages & exp…

0fa2cdc

…orer tabs in Windows 11 + code refactoring

[pre-commit.ci] auto fixes from pre-commit.com hooks

91b5de0

for more information, see https://pre-commit.ci

lunixbochs reviewed Sep 8, 2025

View reviewed changes

nriley requested changes Sep 13, 2025

View reviewed changes

nriley and others added 4 commits September 13, 2025 20:36

Merge branch 'main' into main-upstream

d9f550f

Merge branch 'main' into main-upstream

36f24bd

[pre-commit.ci] auto fixes from pre-commit.com hooks

ad83302

for more information, see https://pre-commit.ci

Merge branch 'main' into main-upstream

1b6cbb8

nriley requested changes Oct 25, 2025

View reviewed changes

nriley added the needs testing Please test this and provide feedback in the PR! label Dec 27, 2025

nriley added 2 commits December 27, 2025 12:41

Merge branch 'main' into main-upstream

0962b2b

Merge branch 'main' into main-upstream

3ece179

knausj85 reviewed Jan 24, 2026

View reviewed changes

		if DEBUG_LOGGING:
		logging.basicConfig(level=logging.DEBUG, format="%(levelname)s: %(message)s")

		@@ -92,14 +475,16 @@ def file_manager_open_parent():

		def file_manager_current_path():
		path = ui.active_window().title

Conversation

xavier-ateo commented Sep 8, 2025

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

xavier-ateo commented Sep 15, 2025

Uh oh!

nriley left a comment • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

xavier-ateo commented Oct 27, 2025

Uh oh!

nriley commented Dec 27, 2025

Uh oh!

knausj85 Jan 24, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

nriley left a comment •

edited

Loading

knausj85 Jan 24, 2026 •

edited

Loading