Skip to content
This repository has been archived by the owner. It is now read-only.

Pie charts bigger than the widget container #1487

Closed
razvanphp opened this issue Jun 15, 2015 · 7 comments
Closed

Pie charts bigger than the widget container #1487

razvanphp opened this issue Jun 15, 2015 · 7 comments
Assignees
Labels

Comments

@razvanphp
Copy link

@razvanphp razvanphp commented Jun 15, 2015

screen shot 2015-06-15 at 18 45 30

Pie chart is always cut by the scroll.

Also I think the values should be displayed on the pie itself since you can turn show data table: false and then you don't know what your chart is showing.

@edmundoa
Copy link
Member

@edmundoa edmundoa commented Jun 17, 2015

Hi,

Could you please indicate the Graylog version you are running? It would also help to know the browser you are using and any errors that you may see in your browser's Javascript console.

@razvanphp
Copy link
Author

@razvanphp razvanphp commented Jun 17, 2015

This happens in both Chrome and latest Firefox. There are no errors in the console.

graylog-web-interface v1.1.1 (893e8e7)

It seems it's fine at the first load, but as soon as the pie is refresh with AJAX it looks like on the screenshot.

@edmundoa
Copy link
Member

@edmundoa edmundoa commented Jun 17, 2015

That issue should be fixed in 1.1.2, could you please give it a try?

@razvanphp
Copy link
Author

@razvanphp razvanphp commented Jun 17, 2015

Yes, looks good, thanks!

@razvanphp razvanphp closed this Jun 17, 2015
@edmundoa
Copy link
Member

@edmundoa edmundoa commented Jun 17, 2015

Good to hear that, welcome! 👍

@razvanphp
Copy link
Author

@razvanphp razvanphp commented Sep 7, 2015

@edmundoa this issue is again present in v1.2.0-rc.2, it should be no scroll there..

screen shot 2015-09-07 at 15 04 04

@edmundoa
Copy link
Member

@edmundoa edmundoa commented Sep 7, 2015

Not sure what changed there, but will take a look.

@edmundoa edmundoa reopened this Sep 7, 2015
@edmundoa edmundoa added the usability label Sep 7, 2015
@edmundoa edmundoa self-assigned this Sep 7, 2015
@edmundoa edmundoa closed this in a817c67 Sep 9, 2015
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
2 participants
You can’t perform that action at this time.