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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

sphinx-mermaid changes my page's CSS, sets "section" opacity to 0.2, results in my page content being semi-transparent. #7

Closed
Zulko opened this issue May 19, 2017 · 10 comments

Comments

@Zulko
Copy link

Zulko commented May 19, 2017

Sorry for the long title. everything is in it 馃槃
The moment I put a .. mermaid:: in my page, the CSS changes and in particular the section tags (opacity: 0.2).
Looking into the code it seems that you are aware of this and there is a workaround in the code, but the pypi version still has this problem apparemtly.

@mgaitan
Copy link
Owner

mgaitan commented May 19, 2017

Yeah, it's an issue in mermaid. I raised a ticket there mermaid-js/mermaid#527. Check my workaround here b2740ea

@Zulko
Copy link
Author

Zulko commented May 19, 2017

As I said, I have seen the workaround in the code, but the version I got from pip doesnt seem to work - is the workaround pushed on PyPI ?

@mgaitan
Copy link
Owner

mgaitan commented May 19, 2017

oh sorry. yes, the hack is part of the tag 0.2, which is the last version released on pypi. But now I note it only apply to .section (the css class applied to div for example) and not for section as tag. Could you confirm this?

@Zulko
Copy link
Author

Zulko commented May 19, 2017

Oh yes indeed. Now I understand the problem: I am using the popular read-the-docs theme which uses section as a class for all the sections of the documentation. As a consequence everything gets the semi-opacity.

@iroller
Copy link
Contributor

iroller commented Jun 2, 2017

Hey @Zulko, did you figure out the way how to apply the workaround when using read-the-docs theme?

@iroller
Copy link
Contributor

iroller commented Jun 2, 2017

@mgaitan removing .body from your workaround would fix the issue for those using read-the-docs theme.

@mgaitan
Copy link
Owner

mgaitan commented Jun 2, 2017

could you send a tiny PR?

@iroller
Copy link
Contributor

iroller commented Jun 3, 2017

@mgaitan of course. #8

@mgaitan
Copy link
Owner

mgaitan commented Jun 4, 2017

fixed in 3baa0b2

@mgaitan mgaitan closed this as completed Jun 4, 2017
@Zulko
Copy link
Author

Zulko commented Aug 1, 2017

Just to confirm it works like a charm now. Thanks very much guys !

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

No branches or pull requests

3 participants