TheChance101 / AAY-chart

A chart library for Compose Multiplatform

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

xAxisData not showing.

mrabbi-hishabee opened this issue · comments

Details:
I use this library in an XML-based project. It works fine but xAxisData data not showing.

Example:
image

Code:
@composable
fun BarChartSample(xAxisData: List, testBarParameters: List) {
xAxisData.toString().logDebug("RABBI")
BarChart(
legendPosition = LegendPosition.DISAPPEAR,
chartParameters = testBarParameters,
barCornerRadius = 2.dp,
gridColor = Color.LightGray,
xAxisData = xAxisData,
isShowGrid = true,
showXAxis = true,
showYAxis = true,
animateChart = true,
showGridWithSpacer = true,

    yAxisStyle = TextStyle(
        fontSize = 12.sp,
        color = Color.Gray,
    ),
    xAxisStyle = TextStyle(
        fontSize = 12.sp,
        color = Color.Gray,
        fontWeight = FontWeight.W400
    ),
    barWidth = 20.dp
)

}

Unfortunately, this is an issue in our library we will fix it in the next release.

@amnah44 could you share list of issues will be fixed in next release and expected date ?

Unfortunately, this is an issue in our library we will fix it in the next release.

by reading code-base, it's obvious that you decide to stop drawing xDataAxis but I don't realize why you should do that (may be there was a bug in 'xAxisDrawing'):

if (showXAxis) {
        if (!isFromBarChart) {
            xAxisDrawing(
                xAxisData = xAxisData,
                textMeasure = textMeasure,
                xAxisStyle = xAxisStyle,
                specialChart = specialChart,
                upperValue = upperValue,
                xRegionWidth = xRegionWidth
            )
        }
    }

where 'isFromBarChart' is always true!

could you please fix it?

@MahmoudMabrok Most of our team members are college students so we fix this issue in our vacation or free time, so I can't give you an expected date for next release but we are trying to fix all this issue here before new year. and thanks for your ask.

@MahmoudMabrok Thanks for your question and I would like to clarify that we are making this class as a base class able to use for all schemes of our charts now and in the future and we are fixing the UI, not the code issue because every team has plans, goals, ideas and decisions.