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

ER diagrams are insensitive to font size config #3032

Open
aalong-tr opened this issue May 4, 2022 · 2 comments
Open

ER diagrams are insensitive to font size config #3032

aalong-tr opened this issue May 4, 2022 · 2 comments
Labels
Contributor needed Status: Approved Is ready to be worked on Type: Enhancement New feature or request

Comments

@aalong-tr
Copy link

aalong-tr commented May 4, 2022

Describe the bug
Some diagram types respect init theme variables that modify font size, but ER diagrams do not respect this configuration

To Reproduce
Link to Live Editor:
class diagram with working font size
er diagram with ignored font size

Steps to reproduce the behavior:

  1. Load the sample ER diagram (or any other Entity Relationship diagram)
  2. Prepend the snippet %%{init:{"themeVariables": { "fontSize": "30px" }}}%% to the first line

Note that, while I have omitted an explicit theme (which may or may not be defined behavior), adding a theme property to the init json does not alter the bug

Expected behavior
The font size of the diagram should noticeably increase. Instead, it remains unchanged.

Desktop (please complete the following information):

  • OS: OSX 12.13.1
  • Browser Firefox
  • Version 99.0.1
@aalong-tr aalong-tr added the bug label May 4, 2022
@aalong-tr aalong-tr changed the title ER diagrams are insensitive to font ER diagrams are insensitive to font size config May 4, 2022
@asesidaa
Copy link

Neither does sequence diagram

@sidharthv96 sidharthv96 transferred this issue from mermaid-js/mermaid-live-editor May 13, 2022
@github-actions github-actions bot added the Status: Triage Needs to be verified, categorized, etc label May 13, 2022
@sidharthv96
Copy link
Member

Transferring as class diagram does have font size specified in svg, but er diagam doesn't.
Out of scope for live editor.

@jgreywolf jgreywolf added Type: Bug / Error Something isn't working or is incorrect and removed bug labels Jan 26, 2023
@jgreywolf jgreywolf added the roadmap items to add to roadmap for auto workflow label Nov 16, 2023
@jgreywolf jgreywolf added Contributor needed Type: Enhancement New feature or request Status: Approved Is ready to be worked on and removed Type: Bug / Error Something isn't working or is incorrect Status: Triage Needs to be verified, categorized, etc roadmap items to add to roadmap for auto workflow labels Dec 6, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Contributor needed Status: Approved Is ready to be worked on Type: Enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

4 participants