Skip to content
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

[Run > Shell plugin] Shell selection improvements #28972

Merged
merged 16 commits into from
Oct 11, 2023

Conversation

htcfreek
Copy link
Collaborator

@htcfreek htcfreek commented Oct 3, 2023

Summary of the Pull Request

⚠ This PR breake existing third party plugins that have ComboBox options! ⚠ (For compatibility reason we could implement old and new code. But this means we have duplicated code.)

This PR improves the shell settings for the Shell plugin.

Changes in this PR:

  • The ComboBox items of a PluginAdditionalOption now support reordering without braking existing user settings.

  • PowerShell 7 has been added.

  • New settings to run CMD, PowerShell and PowerShell 7 in Windows Terminal as console host.

  • Various text improvements.

PR Checklist

Detailed Description of the Pull Request / Additional comments

image
Note: The name of the "Find and run executable file" item has changed after creating the screenshot.

image

Validation Steps Performed

  • Tested settings functionality.
  • Tested the new shell execution modes.
  • Validated that setting don't hang or crash if the json content uses the old notation.

@github-actions

This comment has been minimized.

@htcfreek htcfreek changed the title [Run > Shell plugin] Shell selection improvements 🚧 [Run > Shell plugin] Shell selection improvements Oct 3, 2023
@github-actions

This comment has been minimized.

@htcfreek
Copy link
Collaborator Author

htcfreek commented Oct 3, 2023

⚠ This PR breake existing third party plugins that have ComboBox options! ⚠ (For compatibility reason we could implement old and new code. But this means we have duplicated code.)

@jaimecbernardo , @crutkas
What do you think about this?

@htcfreek htcfreek marked this pull request as ready for review October 3, 2023 13:34
@htcfreek htcfreek self-assigned this Oct 3, 2023
@htcfreek htcfreek added Idea-Enhancement New feature or request on an existing product Run-Plugin Things that relate with PowerToys Run's plugin interface Needs-Review This Pull Request awaits the review of a maintainer. labels Oct 3, 2023
@htcfreek htcfreek changed the title 🚧 [Run > Shell plugin] Shell selection improvements [Run > Shell plugin] Shell selection improvements Oct 3, 2023
@crutkas
Copy link
Member

crutkas commented Oct 3, 2023

How many have combo boxes

@htcfreek
Copy link
Collaborator Author

htcfreek commented Oct 3, 2023

How many have combo boxes

One that I know. This feature was released with v0.73.0 v0.74.0 .

And it only brakes the ability to select an other value in settings ui. The plugin will still work and the setting won't change.

@github-actions

This comment has been minimized.

@htcfreek
Copy link
Collaborator Author

htcfreek commented Oct 4, 2023

@jaimecbernardo, @crutkas
I made some small improvements for better backward compatibility. This is ready for review.

Copy link
Collaborator

@jaimecbernardo jaimecbernardo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! Thank you for the contribution!
Just tweaked some of the strings. WDYT?

htcfreek and others added 3 commits October 10, 2023 18:43
Copy link
Collaborator

@jaimecbernardo jaimecbernardo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! Thank you for the contribution!
Just merged main in to get CI passes and should merge after CI.

@htcfreek
Copy link
Collaborator Author

@jaimecbernardo
CI passed.

@jaimecbernardo jaimecbernardo merged commit 19827d0 into microsoft:main Oct 11, 2023
@htcfreek htcfreek deleted the PT_PluginShellSelect branch October 11, 2023 13:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Idea-Enhancement New feature or request on an existing product Needs-Review This Pull Request awaits the review of a maintainer. Run-Plugin Things that relate with PowerToys Run's plugin interface
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants