-
-
Notifications
You must be signed in to change notification settings - Fork 25
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
Make 3.0.0 release #48
Comments
As a result of #47 once completed. |
It seems to be completed ;-) |
Latest master available on sandbox for testing and extending by additional examples. The only irritating thing I realize now it the overflow bars which we are unconditionally getting. I believe this needs to be tweaked. @cicalese I think this enhancement was originally created by you and it worked in previous releases, i.e. we got the bars only for oversized charts etc. Perhaps you have an idea as to why this is now shown unconditionally. |
@kghbln I did not see the scrollbars in the examples, but they are probably due to
which as introduced into MediaWiki:Common.css in https://sandbox.semantic-mediawiki.org/w/index.php?title=MediaWiki:Common.css&diff=prev&oldid=47241 due to #41. If you don't want to add that unconditionally, maybe you could use TemplateStyles to add it only to pages that need it? |
Hmm, probably a browser related issue. This is what Fx shows:
Ah, I figured this was part of the CSS shipped with the extension. That explains it. Thanks a lot for the pointer.
No, since the latest release it really looks ugly, so an alternative solution probably involving a template using TemplateStyles is the best option here. Thanks again for the suggestion! |
The text was updated successfully, but these errors were encountered: