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

Simplify the sequence/message flow icon #2282

Closed
eriknyk opened this issue May 21, 2021 · 8 comments · Fixed by #3430
Closed

Simplify the sequence/message flow icon #2282

eriknyk opened this issue May 21, 2021 · 8 comments · Fixed by #3430
Assignees
Labels
a11y modeling spring cleaning Could be cleaned up one day
Milestone

Comments

@eriknyk
Copy link

eriknyk commented May 21, 2021

What should we do?

Simplify the connect icon from

connect-tool-screenshot

to

single arrow

Why should we do it?

Improve the icons accessibility ⬇️.


What

Please, change the sequence/message flow icon

From:

connect-tool-screenshot

By the single version of both sequence line + arrow, like

single arrow

and

Screen Shot 2021-05-21 at 11 08 57

I think that by default can be the first image, and change it on demand later (with a context menu similar when you switch from task types ) to dotted arrow.

Why?

There are some people (like me) that have a weird eye condition where with icons like that,

image

It causes that the brain thinks that the image is not focused and it tries to focus but since the image is ok and will not change the brain tries to focus all time causing a fatal tiredness and even headache.

I'm pretty sure that I'm not the only one with this problem since the doctor told me that is a normal issue for some people with certain age.

Best Regards, and congrats for all you awesome work.

@nikku
Copy link
Member

nikku commented May 25, 2021

Thanks for opening this issue.

What the icon should transport is make a connection. We want to avoid that users think only a single connection type is possible. We also want to avoid to show more than one entry in the context pad and in the palette.

Do you have a suggestion how we could improve the icon with regards to your issue while continuing to transport that message?

@nikku nikku added help wanted Extra attention is needed a11y modeling labels May 25, 2021
@nikku
Copy link
Member

nikku commented Jun 2, 2021

Would be interesting to get your feedback on ⬆️ @eriknyk.

@eriknyk
Copy link
Author

eriknyk commented Jun 3, 2021

Thanks for you answer @nikku, I was thinking a way to try minimize the effect, and something like this can help.

copy

I didn't try other colors, but yeah it minimize the effect, not at 100% but at least about 75% what is acceptable helpful.

@andreasgeier
Copy link

This symbol does not only represent (a standard) connection but also message flow, conditional flow, and default flow. It is a symbol that represents an entire group like most of our other symbols in the palette and context pad do (gateway, task, end event, etc.).

So my argument would be that one representative symbol is enough. In this case, I would choose the standard connection:
symbol

@nikku nikku added backlog Queued in backlog spring cleaning Could be cleaned up one day and removed help wanted Extra attention is needed labels Jun 14, 2021
@nikku nikku changed the title Change the sequence/message flow icon Simplify the sequence/message flow icon Jun 14, 2021
@nikku
Copy link
Member

nikku commented Jun 14, 2021

Your rationale @andreasgeier makes absolute sense.

I've updated the issue description accordingly. We'll give the simplified connection symbol (sequence flow style only) a try.

@eriknyk
Copy link
Author

eriknyk commented Sep 16, 2022

@nikku just checking in on this one to see if there is any update.

Regards.

@nikku
Copy link
Member

nikku commented Sep 16, 2022

We did not pick it up yet.

@barmac barmac self-assigned this Jan 30, 2023
barmac added a commit to bpmn-io/bpmn-js that referenced this issue Jan 30, 2023
barmac added a commit to bpmn-io/bpmn-font that referenced this issue Jan 31, 2023
barmac added a commit to bpmn-io/bpmn-font that referenced this issue Jan 31, 2023
nikku pushed a commit to bpmn-io/bpmn-font that referenced this issue Jan 31, 2023
nikku pushed a commit to bpmn-io/bpmn-font that referenced this issue Jan 31, 2023
nikku added a commit to bpmn-io/bpmn-js that referenced this issue Feb 1, 2023
nikku added a commit to bpmn-io/bpmn-js that referenced this issue Feb 1, 2023
@barmac
Copy link
Collaborator

barmac commented Feb 2, 2023

This is now fixed upstream via bpmn-io/bpmn-font#19

@barmac barmac added fixed upstream Requires integration of upstream change and removed backlog Queued in backlog labels Feb 2, 2023
barmac added a commit that referenced this issue Feb 8, 2023
@bpmn-io-tasks bpmn-io-tasks bot added the in progress Currently worked on label Feb 8, 2023
@bpmn-io-tasks bpmn-io-tasks bot removed the fixed upstream Requires integration of upstream change label Feb 8, 2023
barmac added a commit that referenced this issue Feb 8, 2023
barmac added a commit that referenced this issue Feb 8, 2023
@bpmn-io-tasks bpmn-io-tasks bot added needs review Review pending and removed in progress Currently worked on labels Feb 8, 2023
barmac added a commit that referenced this issue Feb 8, 2023
@bpmn-io-tasks bpmn-io-tasks bot removed the needs review Review pending label Feb 8, 2023
@nikku nikku added this to the M61 milestone Feb 8, 2023
lzgabel pushed a commit to lzgabel/camunda-modeler that referenced this issue Apr 4, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
a11y modeling spring cleaning Could be cleaned up one day
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants