Skip to content

Conversation

themike10452
Copy link
Contributor

@themike10452 themike10452 commented Aug 22, 2021

add run as administration when holding ctrl and shift to Program and Explorer plugins.

change Shell plugin's admin key from ctrl to ctrl and shift

Copy link
Member

@taooceros taooceros left a comment

Choose a reason for hiding this comment

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

Thanks for contribution!

@jjw24
Copy link
Member

jjw24 commented Aug 22, 2021

Thanks @themike10452

What about changing Ctrl + Shift to be just Ctrl? So it's the same as Shell plugin invoking administrator.

@taooceros taooceros added the enhancement New feature or request label Aug 24, 2021
@taooceros taooceros added this to the 1.9.0 milestone Aug 24, 2021
@themike10452
Copy link
Contributor Author

@jjw24 i wanted to mimic the behavior of windows start menu where ctrl+shift+enter will run the selected program as administrator. Microsoft's PowerToys Run uses the same key combination as well.

@jjw24
Copy link
Member

jjw24 commented Aug 24, 2021

@jjw24 i wanted to mimic the behavior of windows start menu where ctrl+shift+enter will run the selected program as administrator. Microsoft's PowerToys Run uses the same key combination as well.

Yeah ok I didn't know that's the windows default.

@taooceros worth realign the default admin key combo for shell plugin?

@taooceros
Copy link
Member

@jjw24 i wanted to mimic the behavior of windows start menu where ctrl+shift+enter will run the selected program as administrator. Microsoft's PowerToys Run uses the same key combination as well.

Yeah ok I didn't know that's the windows default.

@taooceros worth realign the default admin key combo for shell plugin?

@themike10452 Do you want to handle that in this pr together?

@themike10452
Copy link
Contributor Author

@jjw24 i wanted to mimic the behavior of windows start menu where ctrl+shift+enter will run the selected program as administrator. Microsoft's PowerToys Run uses the same key combination as well.

Yeah ok I didn't know that's the windows default.
@taooceros worth realign the default admin key combo for shell plugin?

@themike10452 Do you want to handle that in this pr together?

sure I'll add that to the PR. I wanted to look at launching UWP apps as admin as well.

@themike10452
Copy link
Contributor Author

@jjw24 I added 2 commits to this PR, we can now launch UWP apps and shell commands as admin when holding ctrl + shift.
I also added a run as admin option to the context menu of UWP app results.
Programs must be reindexed before they can run as admin.

@taooceros
Copy link
Member

Amazing, I want to launcher windows terminal as administrative for a long time.

@themike10452
Copy link
Contributor Author

Amazing, I want to launcher windows terminal as administrative for a long time.

me too!

taooceros
taooceros previously approved these changes Sep 12, 2021
Copy link
Member

@taooceros taooceros left a comment

Choose a reason for hiding this comment

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

Will wait for a while to merge for others to take a look

Copy link
Member

@jjw24 jjw24 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 @themike10452. Just added version bumps since we also added admin launch for files in Explorer plugin.

jjw24
jjw24 previously approved these changes Sep 12, 2021
@jjw24
Copy link
Member

jjw24 commented Sep 12, 2021

@taooceros let's hold for 1.9.0.

This pr is available in the pre-release #640

@jjw24 jjw24 merged commit 1a7c172 into Flow-Launcher:dev Sep 23, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants