fix(class): avoid duplicate definition of fill

[Mister-Hope]

📑 Summary

Brief description about the content of your PR.

Resolves #

📏 Design Decisions

Describe the way your implementation works or what design decisions you made if applicable.

📋 Tasks

Make sure you

• 📖 have read the contribution guidelines
• 💻 have added necessary unit/e2e tests.
• 📓 have added documentation. Make sure MERMAID_RELEASE_VERSION is used for all new features.
• 🔖 targeted develop branch

[netlify]

✅ Deploy Preview for mermaid-js ready!

Name	Link
🔨 Latest commit	444e81a
🔍 Latest deploy log	https://app.netlify.com/sites/mermaid-js/deploys/65236edb657c830007fdf097
😎 Deploy Preview	https://deploy-preview-4929--mermaid-js.netlify.app
📱 Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.

---

To edit notification comments on pull requests, go to your Netlify site configuration.

[Mister-Hope]

Also I noticed that there are a lot of theme variables, like attributeBackgroundColorOdd and attributeBackgroundColorEven (for er chart), can only be found at source code.

I would suggesting adding theme in docs, if any core team agrees, I can add them in another pr to make sure they are available at docs.

The only question is that shall I add them at the bottom of chart page, or inside config section about themeVariables

[codecov]

Codecov Report

Merging #4929 (444e81a) into develop (12a4707) will not change coverage.
The diff coverage is n/a.

Additional details and impacted files

@@           Coverage Diff            @@
##           develop    #4929   +/-   ##
========================================
  Coverage    79.97%   79.97%           
========================================
  Files          164      164           
  Lines        13623    13623           
  Branches       693      693           
========================================
  Hits         10895    10895           
  Misses        2579     2579           
  Partials       149      149           

Flag	Coverage Δ
e2e	85.26% <ø> (ø)
unit	43.29% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Files	Coverage Δ
packages/mermaid/src/diagrams/class/styles.js	100.00% <ø> (ø)

[huynhicode]

Also I noticed that there are a lot of theme variables, like attributeBackgroundColorOdd and attributeBackgroundColorEven (for er chart), can only be found at source code.

I would suggesting adding theme in docs, if any core team agrees, I can add them in another pr to make sure they are available at docs.

The only question is that shall I add them at the bottom of chart page, or inside config section about themeVariables

Thanks @Mister-Hope

Perhaps you can add them in these two places:

• https://mermaid.js.org/syntax/entityRelationshipDiagram.html#styling
• https://mermaid.js.org/config/theming.html#theme-variables

[mermaid-bot]

@Mister-Hope, Thank you for the contribution!
You are now eligible for a year of Premium account on MermaidChart.
Sign up with your GitHub account to activate.