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

feat: use forest theme in diagram #752

Open
wants to merge 33 commits into
base: master
Choose a base branch
from
Open

Conversation

Yash-Singh1
Copy link
Member

📑 Summary

Use the forest theme provided by mermaid when the theme is set to forest on the platform.

📏 Design Decisions

Added a bit more logic checking if the theme is forest in toggleDarkTheme in the diagram.

📋 Tasks

Make sure you

  • 📖 have read the contribution guidelines
  • 💻 have added unit/e2e tests (if appropriate)
  • 🔖 targeted develop branch

@netlify
Copy link

netlify bot commented Apr 17, 2022

Deploy Preview for mermaidjs ready!

Name Link
🔨 Latest commit 010a6d7
🔍 Latest deploy log https://app.netlify.com/sites/mermaidjs/deploys/62ab7f2f77dcfd000853cbf5
😎 Deploy Preview https://deploy-preview-752--mermaidjs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

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

@cypress
Copy link

cypress bot commented Apr 17, 2022



Test summary

24 0 1 0


Run details

Project Mermaid Live Editor
Status Passed
Commit a1ebb7b ℹ️
Started Jun 16, 2022 7:07 PM
Ended Jun 16, 2022 7:10 PM
Duration 02:42 💡
OS Linux Ubuntu - 20.04
Browser Electron 100

View run in Cypress Dashboard ➡️


This comment has been generated by cypress-bot as a result of this project's GitHub integration settings. You can manage this integration in this project's settings in the Cypress Dashboard

src/lib/util/state.ts Outdated Show resolved Hide resolved
src/lib/util/state.ts Outdated Show resolved Hide resolved
Co-authored-by: Sidharth Vinod <github@sidharth.dev>
@sidharthv96
Copy link
Member

sidharthv96 commented Apr 22, 2022

It does look very nice with the theme, but I'm not able to see the connecting lines...
Maybe we should introduce a forest-dark in mermaid with white lines?

New
image

Old
image

We're losing a lot of info on other diagram types too
image
image
image

@Yash-Singh1
Copy link
Member Author

Yash-Singh1 commented Apr 22, 2022

Maybe we should introduce a forest-dark in mermaid with white lines?

Sure, that sounds good.

sidharthv96 and others added 23 commits June 2, 2022 13:07
Bumps [eslint-plugin-mocha](https://github.com/lo1tuma/eslint-plugin-mocha) from 10.0.4 to 10.0.5.
- [Release notes](https://github.com/lo1tuma/eslint-plugin-mocha/releases)
- [Changelog](https://github.com/lo1tuma/eslint-plugin-mocha/blob/master/CHANGELOG.md)
- [Commits](lo1tuma/eslint-plugin-mocha@10.0.4...10.0.5)

---
updated-dependencies:
- dependency-name: eslint-plugin-mocha
  dependency-type: direct:development
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <support@github.com>
Bumps [postcss-load-config](https://github.com/postcss/postcss-load-config) from 4.0.0 to 4.0.1.
- [Release notes](https://github.com/postcss/postcss-load-config/releases)
- [Changelog](https://github.com/postcss/postcss-load-config/blob/main/CHANGELOG.md)
- [Commits](postcss/postcss-load-config@v4.0.0...v4.0.1)

---
updated-dependencies:
- dependency-name: postcss-load-config
  dependency-type: direct:development
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <support@github.com>
Bumps [cypress-localstorage-commands](https://github.com/javierbrea/cypress-localstorage-commands) from 1.7.0 to 2.1.0.
- [Release notes](https://github.com/javierbrea/cypress-localstorage-commands/releases)
- [Changelog](https://github.com/javierbrea/cypress-localstorage-commands/blob/master/CHANGELOG.md)
- [Commits](javierbrea/cypress-localstorage-commands@v1.7.0...v2.1.0)

---
updated-dependencies:
- dependency-name: cypress-localstorage-commands
  dependency-type: direct:development
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <support@github.com>
Bumps [husky](https://github.com/typicode/husky) from 7.0.4 to 8.0.1.
- [Release notes](https://github.com/typicode/husky/releases)
- [Commits](typicode/husky@v7.0.4...v8.0.1)

---
updated-dependencies:
- dependency-name: husky
  dependency-type: direct:development
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <support@github.com>
…op/husky-8.0.1

chore(deps-dev): bump husky from 7.0.4 to 8.0.1
…op/cypress-localstorage-commands-2.1.0

chore(deps-dev): bump cypress-localstorage-commands from 1.7.0 to 2.1.0
…op/postcss-load-config-4.0.1

chore(deps-dev): bump postcss-load-config from 4.0.0 to 4.0.1
…op/eslint-plugin-mocha-10.0.5

chore(deps-dev): bump eslint-plugin-mocha from 10.0.4 to 10.0.5
Bumps [@sveltejs/adapter-static](https://github.com/sveltejs/kit/tree/HEAD/packages/adapter-static) from 1.0.0-next.32 to 1.0.0-next.34.
- [Release notes](https://github.com/sveltejs/kit/releases)
- [Changelog](https://github.com/sveltejs/kit/blob/master/packages/adapter-static/CHANGELOG.md)
- [Commits](https://github.com/sveltejs/kit/commits/@sveltejs/adapter-static@1.0.0-next.34/packages/adapter-static)

---
updated-dependencies:
- dependency-name: "@sveltejs/adapter-static"
  dependency-type: direct:development
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <support@github.com>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Bumps [lint-staged](https://github.com/okonet/lint-staged) from 13.0.0 to 13.0.1.
- [Release notes](https://github.com/okonet/lint-staged/releases)
- [Commits](lint-staged/lint-staged@v13.0.0...v13.0.1)

---
updated-dependencies:
- dependency-name: lint-staged
  dependency-type: direct:development
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <support@github.com>
Bumps [eslint-plugin-tailwindcss](https://github.com/francoismassart/eslint-plugin-tailwindcss) from 3.5.0 to 3.5.2.
- [Release notes](https://github.com/francoismassart/eslint-plugin-tailwindcss/releases)
- [Commits](francoismassart/eslint-plugin-tailwindcss@v3.5.0...v3.5.2)

---
updated-dependencies:
- dependency-name: eslint-plugin-tailwindcss
  dependency-type: direct:development
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <support@github.com>
dependabot bot and others added 8 commits June 10, 2022 07:08
Bumps node from 18.2.0 to 18.3.0.

---
updated-dependencies:
- dependency-name: node
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>
…e-18.3.0

chore(deps): bump node from 18.2.0 to 18.3.0
…op/eslint-plugin-tailwindcss-3.5.2

chore(deps-dev): bump eslint-plugin-tailwindcss from 3.5.0 to 3.5.2
…op/lint-staged-13.0.1

chore(deps-dev): bump lint-staged from 13.0.0 to 13.0.1
Bumps [tailwindcss](https://github.com/tailwindlabs/tailwindcss) from 3.0.24 to 3.1.1.
- [Release notes](https://github.com/tailwindlabs/tailwindcss/releases)
- [Changelog](https://github.com/tailwindlabs/tailwindcss/blob/master/CHANGELOG.md)
- [Commits](tailwindlabs/tailwindcss@v3.0.24...v3.1.1)

---
updated-dependencies:
- dependency-name: tailwindcss
  dependency-type: direct:development
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>
…op/tailwindcss-3.1.1

chore(deps-dev): bump tailwindcss from 3.0.24 to 3.1.1
@github-actions
Copy link

This pr is stale because it has been open 90 days with no activity. Remove stale label or comment or this will be closed in 30 days

@github-actions github-actions bot added the Stale label Sep 15, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants