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

Reformatting of css files #331

Merged
merged 4 commits into from
Mar 29, 2016
Merged

Reformatting of css files #331

merged 4 commits into from
Mar 29, 2016

Conversation

Jmuccigr
Copy link
Contributor

A simple reformatting of the css files with no changes to content. The reformatting was done with BBEdit.

@knsv
Copy link
Collaborator

knsv commented Mar 22, 2016

Thank you for your effort! I do however see a problem with your changes as these files are generated from the less sources.

See the files in the folder:
https://github.com/knsv/mermaid/tree/master/src/less

Take a look on those and see if any of your changes should be applied. If I merge this your changes will be overwritten during next build.

I am sorry about that but I hope that some of your ideas can be moved.

Good luck,

@Jmuccigr
Copy link
Contributor Author

Thanks. It occurred to me after I had pushed this that these files might be auto generated. I appreciate the note. I'm a little in the dark about where the various css files come from. For example, I don't seed the mono-blue one in the src dir.

@knsv
Copy link
Collaborator

knsv commented Mar 29, 2016

Will merge but dont get disappointed when things overwritten.

@knsv knsv merged commit 9e49617 into mermaid-js:master Mar 29, 2016
@Jmuccigr
Copy link
Contributor Author

Well, I'd rather contribute something helpful. :-) Where do the various css files come from in the source, as I wrote above? For example, mono-blu?

mgenereu pushed a commit to mgenereu/mermaid that referenced this pull request Jun 25, 2022
…yarn/develop/svelte-3.42.4

chore(deps-dev): bump svelte from 3.42.3 to 3.42.4
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