Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Faces in Journey Diagram Out of Bound #6262

Open
megantriplett opened this issue Feb 7, 2025 · 1 comment · May be fixed by #6263
Open

Faces in Journey Diagram Out of Bound #6262

megantriplett opened this issue Feb 7, 2025 · 1 comment · May be fixed by #6263
Labels
Status: Triage Needs to be verified, categorized, etc Type: Bug / Error Something isn't working or is incorrect

Comments

@megantriplett
Copy link

Description

When I attempt to use a value greater than 7 for a face position in a Journey Diagram, the smiley face appears over the horizontal dividing line and sometimes within the boxes.

Steps to reproduce

Type a number greater than 7 after the event in a journey diagram, such as the 12 below:
event: 12: person

Screenshots

Image

Code Sample

journey
title Our Day
section Morning
Breakfast:10: Megan
Stretch : 11: Udval
Brush Teeth:9: Monica
section Class
go to class: 13 : Udval

Setup

  • Mermaid version: 11.4.1
  • Browser and Version: Firefox

Suggested Solutions

An alert when an invalid value is entered and the value is reset to the maximum

Additional Context

No response

@megantriplett megantriplett added Status: Triage Needs to be verified, categorized, etc Type: Bug / Error Something isn't working or is incorrect labels Feb 7, 2025
@megantriplett
Copy link
Author

@megantriplett @udvale and @nghtlinh are working on this and are almost done writing the test cases

@nghtlinh nghtlinh linked a pull request Feb 7, 2025 that will close this issue
4 tasks
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
Status: Triage Needs to be verified, categorized, etc Type: Bug / Error Something isn't working or is incorrect
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant