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

Colors are not properly exposed when the number of data.series is more than 27 #554

Closed
devwoo725 opened this issue Feb 1, 2021 · 3 comments

Comments

@devwoo725
Copy link

devwoo725 commented Feb 1, 2021

Describe the bug
pie chart에서 data.series 개수가 27개 이상일 경우 에러 메시지가 노출되고 차트가 그려지지 않습니다.
(에러 메시지는 아래에 복사해 두었습니다.)


translated

If the number of data.series in the pie chart is more than 27, an error message is displayed and the chart is not drawn.
(The error message has been copied below.)

To Reproduce
Steps to reproduce the behavior:
아래 소스 실행하면 됩니다.(Just run the source below.)

<head>
    <title>Title</title>
    <link rel="stylesheet" href="${pageContext.request.contextPath}/toast-ui-chart/css/toastui-chart.min.css">
    <script type="text/javascript" src="${pageContext.request.contextPath}/toast-ui-chart/toastui-chart.min.js"></script>
</head>
<body>
<div id="chart-area"></div>
<script type="text/javascript">
    let el = document.getElementById('chart-area');
    const data = {
        series: [
            {
                name: 'Chrome',
                data: 46.02,
            },
            ....26개 더
        ],
    };
    const options = {
        chart: { title: 'Usage share of web browsers', width: 600, height: 600 },
    };
    toastui.Chart.pieChart({ el, data, options });
</script>
</body>

Desktop (please complete the following information):

  • OS: win10 pro
  • Browser [chrome]
  • Version [버전 87.0.4280.141(공식 빌드) (64비트)]

Additional context
toastui-chart.min.js:7 Uncaught TypeError: Cannot read property 'toLowerCase' of undefined
at nt (toastui-chart.min.js:7)
at at (toastui-chart.min.js:7)
at i.value (toastui-chart.min.js:7)
at toastui-chart.min.js:7
at Array.forEach ()
at i.value (toastui-chart.min.js:7)
at i.value (toastui-chart.min.js:7)
at o (toastui-chart.min.js:7)
at e. (toastui-chart.min.js:7)
at t (toastui-chart.min.js:7)

toastui-chart.min.js:7 Uncaught TypeError: Cannot read property 'clearRect' of undefined
at e.value (toastui-chart.min.js:7)
at c.value (toastui-chart.min.js:7)
at toastui-chart.min.js:7
at Array.forEach ()
at e.value (toastui-chart.min.js:7)
at e.value (toastui-chart.min.js:7)
at toastui-chart.min.js:7

@jungeun-cho jungeun-cho self-assigned this Feb 1, 2021
@jung-han jung-han changed the title data.series 개수가 27개 이상일 경우 문제 확인 요청 Colors are not properly exposed when the number of data.series is more than 27 Feb 2, 2021
@jung-han
Copy link
Member

jung-han commented Feb 2, 2021

#555

This issue occurred because the number of colors was limited to about 30. If the number of colors is increased and exceeded, the previous color will be referenced. Thanks for the bug reporting!

(I am sorry that I changed the contents to English at my own pace.)

@jung-han
Copy link
Member

@devwoo725
Resolved in 4.1.0! Check this version!

@stale
Copy link

stale bot commented Mar 20, 2021

This issue has been automatically marked as inactive because there hasn’t been much going on it lately. It is going to be closed after 7 days. Thanks!

@stale stale bot added the inactive label Mar 20, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants