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

Running scripts from Mobile web interface #6033

Open
takuy opened this issue Apr 16, 2024 · 9 comments · Fixed by #6044
Open

Running scripts from Mobile web interface #6033

takuy opened this issue Apr 16, 2024 · 9 comments · Fixed by #6044
Assignees

Comments

@takuy
Copy link

takuy commented Apr 16, 2024

Is your feature request related to a problem? Please describe.
There doesn't seem to be a way to run scripts or preconfiguredscripts from the mobile web interface.

Describe the solution you'd like
Adding a button on the bottom toolbar to run preconfiguredscripts on Mobile, or when connected similar to the keys sending interface.

@si458
Copy link
Collaborator

si458 commented Apr 19, 2024

you mean you just want a copy of the Run Commands but in the mobile ui (when you click the PLAY icon bottom left?)
image

@takuy
Copy link
Author

takuy commented Apr 19, 2024

you mean you just want a copy of the Run Commands but in the mobile ui (when you click the PLAY icon bottom left?) image

That, but also if you right-click the play icon, you get a list of pre-configured scripts. Running pre-configured scripts is a bit more useful in the Mobile UI, to me anyways.

@si458
Copy link
Collaborator

si458 commented Apr 19, 2024

The is already a feature request for the pre-defined run commands.
But I can look at adding the panel above into the mobile ui for u to begin with 👍

@takuy
Copy link
Author

takuy commented Apr 19, 2024

The is already a feature request for the pre-defined run commands. But I can look at adding the panel above into the mobile ui for u to begin with 👍

Can you link me that feature request? I didn't seen an open one.

The feature below is what would be the most helpful to me in the mobile UI, personally. I don't think being able to type out commands on mobile would be very useful (autocorrect with scripting sounds insane lol)
image

@si458
Copy link
Collaborator

si458 commented Apr 20, 2024

Will look soon for u and attached soon.
Also sorry I forgot about the preconfiguredscripts !
You learn something new every day even when u help maintain software u use every day haha

@si458
Copy link
Collaborator

si458 commented Apr 24, 2024

ive merged the PR to add the manual run commands to the mobile ui

im still looking into the adding of the PreConfiguredScripts as you cant right click on a mobile device
im thinking, clicking run button, then from a new dropdown, select PreConfiguredScripts or Manual Scripts
if PreConfiguredScripts then display your scripts, or Manual Scripts just display the normal run commands

would that be ok?

@si458 si458 self-assigned this Apr 24, 2024
@takuy
Copy link
Author

takuy commented Apr 24, 2024

ive merged the PR to add the manual run commands to the mobile ui

im still looking into the adding of the PreConfiguredScripts as you cant right click on a mobile device im thinking, clicking run button, then from a new dropdown, select PreConfiguredScripts or Manual Scripts if PreConfiguredScripts then display your scripts, or Manual Scripts just display the normal run commands

would that be ok?

Sure. That sounds great. Thank you for your effort on this!

@folosleg
Copy link

folosleg commented May 27, 2024

Thanks, very useful feature indeed!
Also can you put corresponding buttons (Chat. Message, Run) on the General veiw page, inline with the desktop ui.
It's a bit fiddly on the bottom corner of the desktop view...
What i mean is something like this:
mctrl_mobile_ui_general_buttons

@si458
Copy link
Collaborator

si458 commented May 27, 2024

@folosleg can u open a separate enhancement issue for that plz?
Just want everything separate so I don't loose track!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants