[Feature]: Allow for BarList to disable sorting

[mattiaz9]

What problem does this feature solve?

I need the BarList to show elements in a specific order.
This is useful for elements that must follow a specific order, for instance energy classification like A+, A, B ...., or grades, or whatever.
The component should not be opinionated about this.

What does the proposed API look like?

Add the option none to the sortOrder prop.

[mattiaz9]

Another possible solutions would be to remove the default value here:

tremor/src/components/vis-elements/BarList/BarList.tsx

Line 42 in 790ed0c

sortOrder = "descending",

[severinlandolt]

Hi @mattiaz9! If you need a quick solution, check out our Raw Version of the component here: https://raw.tremor.so/docs/visualizations/barlist

There you have full control over the sorting, happy coding!

[severinlandolt]

Note:

import React from 'react';

type BarListProps<T> = {
  data?: T[];
  color?: string;
  valueFormatter?: (value: number) => string;
  showAnimation?: boolean;
  onValueChange?: (value: T) => void;
  sortOrder?: "ascending" | "descending" | "none";
  className?: string;
  [key: string]: any;
};

const defaultValueFormatter = (value: number) => value.toString();

function BarListInner<T>(props: BarListProps<T>, ref: React.ForwardedRef<HTMLDivElement>) {
  const {
    data = [],
    color,
    valueFormatter = defaultValueFormatter,
    showAnimation = false,
    onValueChange,
    sortOrder = "descending",
    className,
    ...other
  } = props;

  const Component = onValueChange ? "button" : "div";

  const sortedData = React.useMemo(() => {
    if (sortOrder === "none") {
      return data;
    }
    return [...data].sort((a, b) => {
      return sortOrder === "ascending" ? a.value - b.value : b.value - a.value;
    });
  }, [data, sortOrder]);

  return (
    <div ref={ref} className={className} {...other}>
      {sortedData.map((item, index) => (
        <Component key={index} style={{ color }}>
          {valueFormatter(item.value)}
        </Component>
      ))}
    </div>
  );
}

export default React.forwardRef(BarListInner);

[github-actions]

🎉 This issue has been resolved in version 3.17.0-beta.1 🎉

The release is available on:

• npm package (@beta dist-tag)
• GitHub release

Your semantic-release bot 📦🚀

[github-actions]

🎉 This issue has been resolved in version 3.17.0 🎉

The release is available on:

• npm package (@latest dist-tag)
• GitHub release

Your semantic-release bot 📦🚀