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

move workflow editor actions out of the toolform #18166

Closed
martenson opened this issue May 16, 2024 · 5 comments · Fixed by #18729
Closed

move workflow editor actions out of the toolform #18166

martenson opened this issue May 16, 2024 · 5 comments · Fixed by #18729

Comments

@martenson
Copy link
Member

These belong to the editor header, next to the undo/redo buttons

Screenshot 2024-05-16 at 11 51 12 AM

@ahmedhamidawan
Copy link
Member

Would having tabs in the right panel still make sense? Since, some of these actions are tabs/actions/dropdowns:

workflow_editor_tabs.mp4

@guerler
Copy link
Contributor

guerler commented May 19, 2024

Thanks for creating this ticket, the workflow ui does need reorganization, but instead of moving the workflow actions to the redo/undo buttons it should be the other way around. We also need to consolidate the comments tool bar with the zoom buttons and ideally move it to the bottom, so it does not jump between vertical and horizontal layout. I would even say that the redo/undo buttons should be placed in there. Right now options appear in five different spots which is confusing. We could easily consolidated by using only two horizontal bars i.e. 1) toolbar at the bottom and 2) workflow actions at the upper right.

image

@martenson
Copy link
Member Author

martenson commented May 20, 2024

@ahmedhamidawan I think I like that. I wouldn't use "lint", that is a tech term.

I'd suggest to move out the "edit report" and "cog" contents to the workflow tab and leave only the "save" and "run" with the undo/redo. (extra point for making the header elements "floaty" and thus giving users more canvas)

Screenshot 2024-05-20 at 1 59 54 PM

@guerler The save and run actions are the most important buttons imho. They should stand out and be reachable at all times. The editor should be useful when the right panel is collapsed and the focus should be on its canvas not on the side panels.
As of where the comment actions/zoom controls/etc. should go, I dare to say majority of our interfaces have actionable elements at the top of the page and workflows would imo benefit from the same.


In my mind in an ideal world the workflow editor would have no sidepanels and full width of the canvas would be occupied by the nodes and panels' functionalities folded in. Then one button would add node with a toolpanel search popping up and node details would only pop when selected.

@guerler
Copy link
Contributor

guerler commented May 21, 2024

@martenson I think that's what I am saying. We should have two locations instead of five and consolidate options into those. 1) toolbar (ideally at the bottom) and 2) workflow actions at the upper right. Having the workflow actions merged into the title header is possible but will squeeze the title if too many are added, but that's a secondary issue.

@ElectronicBlueberry
Copy link
Member

ElectronicBlueberry commented Aug 28, 2024

As of where the comment actions/zoom controls/etc. should go, I dare to say majority of our interfaces have actionable elements at the top of the page and workflows would imo benefit from the same.

This is also a common design pattern, which is why I'd like to stick to it. While I can think of many examples of visual editors which have some sort of tools at the top, I can't think of a single example which has it at the bottom.

I'd also argue that grouping them with the zooms controls is a net minus, as those are two very different actions. Zooming does not change the state of the workflow at all, nor does it precede an action which changes the state. I'd again like to refer to common design patterns. While zoom controls can be found in almost any corner of the window, depending on what software you look at, they are never (to my knowledge) grouped with tools/actions, for the reason outlined above.


#18729 enables stripping most of these actions out, and making the left panel quickly toggleable. With this issue in mind, I have the following proposal for the PR:

  • Move run and save out of the right panel into the center, as suggested here.
  • Move Workflow Attributes and Lint to the activity bar.
  • Move Reports to the activity Bar.
  • Have the right panel exclusively be a node inspector.

What I'm unsure about are the remaining actions hidden in the cog drop-down. They may work as an activity, or it could make them too hard to find. I'll give it a try and see how it feels.

@martenson martenson removed their assignment Oct 31, 2024
@ahmedhamidawan ahmedhamidawan linked a pull request Nov 3, 2024 that will close this issue
4 tasks
@ahmedhamidawan ahmedhamidawan removed their assignment Nov 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: To Do, Assigned
Development

Successfully merging a pull request may close this issue.

4 participants