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

making changes for customize colour #4988

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

sshah135
Copy link

  1. Diagram Styles Enhancement: Updated style functions for various diagram types like flowcharts, pies, and more. We introduced parameters for customizable colors, fonts, and stroke widths, allowing greater flexibility in styling.

  2. Error Resolution: Addressed npm dependency conflicts during the installation of the 'khroma' library. Additionally, suggested installing and updating type definitions to resolve TypeScript-related issues.

  3. Code Simplification: Simplified and clarified existing code snippets for better readability. This includes refactoring complex logic into more concise and comprehensible structures.

  4. Feature Additions: Integrated new features, such as customizable colors for different diagram elements, making your diagrams more visually appealing and adaptable to specific preferences.

  5. Git Hook Troubleshooting: Assisted in resolving an issue with a pre-commit Git hook where 'pnpm' was not found in the PATH. Suggested installing 'pnpm' globally and checking the 'husky' configuration.

These changes aim to improve code maintainability, introduce new features, and troubleshoot issues, ensuring a more robust and customizable codebase.

@netlify
Copy link

netlify bot commented Oct 26, 2023

Deploy Preview for mermaid-js failed.

Name Link
🔨 Latest commit 3b837ea
🔍 Latest deploy log https://app.netlify.com/sites/mermaid-js/deploys/653abac0d85d3000090fe9a3

@sidharthv96
Copy link
Member

Please fix all the failing tests.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants