flekschas/piling.js

pileLabelText is overloaded

flekschas opened this issue · 0 comments

flekschas commented

The pileLabelText does two things currently:

  1. Determine whether a text label is shown
  2. Which text label is shown

We should split this into two separate properties. Why? Because pileLabelText: pile => pile.items.length should work just like pileSizeBadge: pile => pile.items.length > 1 but it doesn't and it's not possible to dynamically enable and disable pile text labels.