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

Provide a human readable comment/name for a shape #1476

Closed
darrenj opened this issue Nov 2, 2019 · 6 comments
Closed

Provide a human readable comment/name for a shape #1476

darrenj opened this issue Nov 2, 2019 · 6 comments
Assignees
Labels
UX Design Need UX Design

Comments

@darrenj
Copy link

darrenj commented Nov 2, 2019

Is your feature request related to a problem? Please describe.

Consider this simple set of shapes below

image

The reasoning/logic behind a set of shapes may not be clear. There should be a way to provide a "comment" that appears instead of the Expression which is not very useful in this view. Alternatively the ability to change the title of the shape could be used for this as well.

You could perhaps have a group shape which you can apply a comment to and then collapse the shapes inside to improve readability.

@darrenj darrenj added the Needs-triage A new issue that require triage label Nov 2, 2019
@darrenj
Copy link
Author

darrenj commented Nov 4, 2019

I've since found you can do this on a number of the shapes - however it's super un-discoverable so think this is a UX bug?

@hibrenda
Copy link
Contributor

hibrenda commented Nov 4, 2019

@sangwoohaan This might be a visual enhancement. Any thoughts ?

  • updates nodes title ?
  • updates nodes content instead of expression ?
  • Add "comments" to group nodes and be collapsed ?

@sangwoohaan
Copy link
Contributor

We might want to revisit the design system for action nodes entirely. Sure, we can patch this up with few tweaks, but it will cause bigger problems across the board if we don't approach this holistically. I briefly talked about it with @mareekuh before as well.

@cwhitten cwhitten removed the Needs-triage A new issue that require triage label Nov 5, 2019
@mareekuh
Copy link
Contributor

mareekuh commented Nov 8, 2019

Agreed - would like to have the design of the nodes be relevant to the action they represent vs the same box for all, just with color differentiation..

@mareekuh
Copy link
Contributor

@sangwoohaan - do we have an uber-issue that captures/collects all the node-redesign requests/Ideas. If so can we tag this one to it and if not, can we create one @hibrenda

@sangwoohaan
Copy link
Contributor

referred in #1701

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

No branches or pull requests

5 participants