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

Pie-chart widgets render too big #123

Closed
vitekpetr opened this issue Nov 16, 2021 · 1 comment · Fixed by #124
Closed

Pie-chart widgets render too big #123

vitekpetr opened this issue Nov 16, 2021 · 1 comment · Fixed by #124

Comments

@vitekpetr
Copy link

vitekpetr commented Nov 16, 2021

Hello,

I have installed Icinga 2 dashboard and I have an issue with the pie-chart widgets and labels.
The widgets render too big for the box and the labes are in grey color when they should be in white.

image

There is an error in browser develepment console. Maybe it has something to do with the issue.
image

  • Dashing version: 1.3.7

  • Ruby version: Ruby 2.7.0p0 (2019-12-25 revision 647ee6f091)

  • Version of this project:
    commit 5ed16c9 (HEAD -> master, origin/master, origin/HEAD)
    Author: Daniel dbodky@gmail.com
    Date: Fri Aug 20 21:47:57 2021 +0200
    diff --git a/Dockerfile b/Dockerfile
    index e2d9fc9..9e18a11 100644
    --- a/Dockerfile
    +++ b/Dockerfile
    @@ -1,6 +1,6 @@
    -FROM ruby:buster
    +FROM ruby:2.7-buster
    -MAINTAINER Michael Friedrich michael.friedrich@gmail.com
    +MAINTAINER Daniel Bodky dbodky@gmail.com
    ENV LANG C.UTF-8
    ENV DEBIAN_FRONTEND noninteractive

  • Modifications to this project, if any (git diff):
    icinga2.txt

  • Operating System and version: Ubuntu 20.04.3 LTS

  • Client browser and version: Google Chrome verison 95.0.4638.69

mocdaniel added a commit that referenced this issue Nov 17, 2021
Fixes #123 by introducing a new widget type called Roundchartjs, adjusting the HTML rendering of the widget to correct width and height for the dashboard's widget tiles.
@mocdaniel
Copy link
Owner

Fixed via #124 .

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 a pull request may close this issue.

2 participants