search

frappe / charts

Simple, responsive, modern SVG Charts with zero dependencies
https://frappe.io/charts
MIT License
14.94k stars 718 forks source link

Labels overlap #171

Closed massimo-cassandro closed 5 years ago

massimo-cassandro commented 6 years ago

Labels in pie charts overlap (even with not very long text). See screenshot

Steps to Reproduce:

{
    "height": 250,
    "title": "Fatturati totali per network",
    "type": "pie",
    "data": {
        "labels": ["Cisalpina Tours", "Bluvacanze", "Travelbuy", "Primarete Viaggi e Vacanze", "Blunet", "Uniposte"],
        "colors": ["#7ED321", "#007bff", "#c00", "#fc0", "#FDA5EE", "#9c6d1e"],
        "datasets": [{
            "name": "ago 2018",
            "values": [3848, 3415, 3500, 2704, 1295, 3339]
        }]
    }
}

schermata 2018-05-07 alle 12 01 10

Frappé Charts version: 1.1.0

ssuess commented 5 years ago

is there a solution to this? I have the same problem...

mojtabaahn commented 5 years ago

same

scmmishra commented 5 years ago

Fixed this in the recent release... labels will automatically fit the width of the chart. (ref: https://github.com/frappe/charts/commit/94c835115b946bad3dc383023c8a8bcb0abe73e2)

Also a I've sent a PR will allow truncating of labels. PR: https://github.com/frappe/charts/pull/234

mojtabaahn commented 5 years ago

@scmmishra thanks so it's gonna be available on v1.2.4 right?

scmmishra commented 5 years ago

The PR has been merged.

so it's gonna be available on v1.2.4 right?

Yes, along with a few other fixes. I am hoping to release it soon.