Skip to content

fix: use ShowItemInFolder for devtools showItemInFolder embedder message#50905

Merged
MarshallOfSound merged 1 commit intomainfrom
fix-devtools-show-item-in-folder
Apr 11, 2026
Merged

fix: use ShowItemInFolder for devtools showItemInFolder embedder message#50905
MarshallOfSound merged 1 commit intomainfrom
fix-devtools-show-item-in-folder

Conversation

@MarshallOfSound
Copy link
Copy Markdown
Member

Use platform_util::ShowItemInFolder (reveal-only) instead of platform_util::OpenPath for the DevTools showItemInFolder embedder message, and gate on registered workspace filesystem paths to match the IsDevToolsFileSystemAdded pattern used elsewhere.

Notes: none

@MarshallOfSound MarshallOfSound added semver/patch backwards-compatible bug fixes target/39-x-y PR should also be added to the "39-x-y" branch. target/40-x-y PR should also be added to the "40-x-y" branch. target/41-x-y PR should also be added to the "41-x-y" branch. target/42-x-y PR should also be added to the "42-x-y" branch. labels Apr 10, 2026
@electron-cation electron-cation bot added the new-pr 🌱 PR opened recently label Apr 10, 2026
@MarshallOfSound MarshallOfSound merged commit b4e14a9 into main Apr 11, 2026
147 of 150 checks passed
@release-clerk
Copy link
Copy Markdown

release-clerk bot commented Apr 11, 2026

No Release Notes

@MarshallOfSound MarshallOfSound deleted the fix-devtools-show-item-in-folder branch April 11, 2026 07:43
@trop
Copy link
Copy Markdown
Contributor

trop bot commented Apr 11, 2026

I was unable to backport this PR to "39-x-y" cleanly;
you will need to perform this backport manually.

@trop trop bot added needs-manual-bp/39-x-y and removed target/39-x-y PR should also be added to the "39-x-y" branch. labels Apr 11, 2026
@trop
Copy link
Copy Markdown
Contributor

trop bot commented Apr 11, 2026

I was unable to backport this PR to "40-x-y" cleanly;
you will need to perform this backport manually.

@trop
Copy link
Copy Markdown
Contributor

trop bot commented Apr 11, 2026

I have automatically backported this PR to "41-x-y", please check out #50937

@trop
Copy link
Copy Markdown
Contributor

trop bot commented Apr 11, 2026

I have automatically backported this PR to "42-x-y", please check out #50938

@trop trop bot added in-flight/42-x-y and removed target/41-x-y PR should also be added to the "41-x-y" branch. target/42-x-y PR should also be added to the "42-x-y" branch. labels Apr 11, 2026
@trop trop bot added merged/42-x-y PR was merged to the "42-x-y" branch. merged/41-x-y PR was merged to the "41-x-y" branch. and removed in-flight/42-x-y in-flight/41-x-y labels Apr 11, 2026
MarshallOfSound added a commit that referenced this pull request Apr 16, 2026
MarshallOfSound added a commit that referenced this pull request Apr 16, 2026
@trop
Copy link
Copy Markdown
Contributor

trop bot commented Apr 16, 2026

@MarshallOfSound has manually backported this PR to "40-x-y", please check out #51114

@trop
Copy link
Copy Markdown
Contributor

trop bot commented Apr 16, 2026

@MarshallOfSound has manually backported this PR to "39-x-y", please check out #51115

codebytere pushed a commit that referenced this pull request Apr 17, 2026
…age (#51114)

fix: use ShowItemInFolder for devtools showItemInFolder embedder message (#50905)

(cherry picked from commit b4e14a9)
@trop trop bot added merged/40-x-y PR was merged to the "40-x-y" branch. and removed in-flight/40-x-y labels Apr 17, 2026
codebytere pushed a commit that referenced this pull request Apr 17, 2026
…age (#51115)

fix: use ShowItemInFolder for devtools showItemInFolder embedder message (#50905)

(cherry picked from commit b4e14a9)
@trop trop bot added merged/39-x-y PR was merged to the "39-x-y" branch. and removed in-flight/39-x-y labels Apr 17, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

merged/39-x-y PR was merged to the "39-x-y" branch. merged/40-x-y PR was merged to the "40-x-y" branch. merged/41-x-y PR was merged to the "41-x-y" branch. merged/42-x-y PR was merged to the "42-x-y" branch. new-pr 🌱 PR opened recently semver/patch backwards-compatible bug fixes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants