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

Problem with Webminstats #1

Closed
Feliksas opened this issue Sep 29, 2014 · 9 comments
Closed

Problem with Webminstats #1

Feliksas opened this issue Sep 29, 2014 · 9 comments

Comments

@Feliksas
Copy link

Greetings,
There's a problem with displaying Webminstats panorama, while this theme is active.
Should be:
http://3.bp.blogspot.com/-KHEcs0TlcIE/U_pUN44P0yI/AAAAAAAAULM/sFXVnMev2So/s1600/Screen%2BShot%2B2014-08-24%2Bat%2B22.06.18.png

Actual situation:
https://yadi.sk/i/S1x1sKBebiTVz

This module is 3rd-party, but can be installed from within Webmin itself (Webmin Modules/Install/Third party module from: "webminstat")

@iliajie
Copy link
Collaborator

iliajie commented Sep 30, 2014

I have checked Webminstats module. It's true that there is no images in panorama. Even though graphs are shown just fine, as you later click on the correspondent link.

There is a problem fixing that - because, it's not really a problem of Authentic.

Webminstats builds/draws graphs/images based on default theme engine, which is totally wrong. Webminstats relies on one part of the code, from the old themes, that I have just intentionally removed.

I would suggest referring with this problem to the developers and asking them to change the way their module interacts with the theme. The way it's build now, is not very flexible/right and should be fixed for preventing future incompatibility issues.

There is not much I can do here. I'm sorry, pal.

@iliajie iliajie closed this as completed Sep 30, 2014
@iliajie
Copy link
Collaborator

iliajie commented Jan 2, 2015

Added basic support for this module. Will be supported in Authentic Theme v.8.0.0.

@gerbier
Copy link

gerbier commented Jan 23, 2015

I am the author of webminstats module. I am using the icons_table call (from core api) to display aligned graphs, what's wrong with this ? (I had a look at others modules, and it seems to common).
Can you provide a documentation link ?

@iliajie
Copy link
Collaborator

iliajie commented Jan 23, 2015

Hello! Thanks for contacting.

I'm not using the part of the code that generates icons. But in the recent version I have made a work around for your module.

You can see the way theme works by simply installing it and looking at authentic.pl.

@gerbier
Copy link

gerbier commented Jan 24, 2015

Hello,

Yes, I can install it on an ubuntu 14.10.
And yesterday, I had a look at ui-lib (which does not exists when webminstats was first coded) :
in next release, I will try to replace my code by ui_* functions. It should work better with themes.

----- Mail original -----

Hello! Thanks for contacting.

I'm not using the part of the code that generates icons. But in the
recent version I have made a work around for your module.

You can see the way theme works by simply installing it and looking
at authentic.pl .


Reply to this email directly or view it on GitHub .

@iliajie
Copy link
Collaborator

iliajie commented Jan 24, 2015

Great, thanks!

@gerbier
Copy link

gerbier commented Apr 9, 2015

Hello

I just release today a new version of webminstats (2.11) which works well with authentic theme, without any workaround in your code.
You will be able to remove this code when you want.

regards,

----- Mail original -----

Hello! Thanks for contacting.

I'm not using the part of the code that generates icons. But in the
recent version I have made a work around for your module.

You can see the way theme works by simply installing it and looking
at authentic.pl .


Reply to this email directly or view it on GitHub .

@iliajie
Copy link
Collaborator

iliajie commented Apr 9, 2015

Perfect, I'll check, thanks, pal!

@iliajie
Copy link
Collaborator

iliajie commented Nov 14, 2017

@gerbier Please take a look: #889 (comment)

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