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

[Cosmetics] separate "Run/Stop on Jenkins" by splitter #461

Closed
jacekgajek opened this issue Feb 27, 2023 · 4 comments
Closed

[Cosmetics] separate "Run/Stop on Jenkins" by splitter #461

jacekgajek opened this issue Feb 27, 2023 · 4 comments

Comments

@jacekgajek
Copy link

image

Action

I click on "Stop On Jenkins"

Expected

Logs are shown

Actual

"Stop On Jenkins" is executed

I can see my future self clicking this accidentally.
Suggestion: Move "Stop on Jenkins" to the last position, separated by a splitter

@MCMicS
Copy link
Owner

MCMicS commented Feb 27, 2023

Thanks for Feedback
Sounds great to move the action.

But the description says that it is expected to showq logs if you click on "Stop on Jenkins". I think this is just a typo isn't?

@jacekgajek
Copy link
Author

It was a joke :)

@MCMicS
Copy link
Owner

MCMicS commented Jul 24, 2023

I keep this at this position because in other menus the Run and Stop action are at same positions (like run from Services view). But I will add a additional separator between Run/Stop and the log actions

@MCMicS MCMicS changed the title [Cosmetics] Move "Stop on Jenkins" to the last position, separate by splitter [Cosmetics] separate "Run/Stop on Jenkins" by splitter Jul 24, 2023
@MCMicS MCMicS closed this as completed in b502d25 Jul 24, 2023
@MCMicS
Copy link
Owner

MCMicS commented Jul 24, 2023

released with 0.13.18

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

No branches or pull requests

2 participants