createProgressBar options

[chimnayajith]

Changes

Additional two options for played time ( leftChar ) and for unplayed time (rightChar) . As it is for the line it goes to ▬ when not provided

Status

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

1 Ignored Deployment

Name	Status	Preview	Comments	Updated (UTC)
discord-player-website	⬜️ Ignored (Inspect)			Jul 5, 2023 1:47am

[twlite]

could we add an option for \u2503 as well?

[chimnayajith]

yea so then it could be

createProgressBar(options) {
  const timestamp = this.getTimestamp();
  if (!timestamp)
    return null;
  const { indicator = "\u{1F518}", leftChar = "\u25AC", rightChar = "\u25AC", length = 15, timecodes = true, separator = "\u2503" } = options || {};
  if (isNaN(length) || length < 0 || !Number.isFinite(length)) {
    throw Exceptions.ERR_OUT_OF_RANGE("[PlayerProgressBarOptions.length]", String(length), "0", "Finite Number");
  }
  const index = Math.round(timestamp.current.value / timestamp.total.value * length);
  if (index >= 1 && index <= length) {
    const bar = leftChar.repeat(index - 1).split("");
    bar.push(indicator);
    bar.push(rightChar.repeat(length - index));
    if (timecodes) {
      return `${timestamp.current.label} ${separator} ${bar.join("")} ${separator} ${timestamp.total.label}`;
    } else {
      return `${bar.join("")}`;
    }
  } else {
    if (timecodes) {
      return `${timestamp.current.label} ${separator} ${indicator}${rightChar.repeat(length - 1)} ${separator} ${timestamp.total.label}`;
    } else {
      return `${indicator}${rightChar.repeat(length - 1)}`;
    }
  }
}

[twlite]

Looks like formatting is not done, could you run yarn format from the root?

[twlite]

Suggested change

	rightChar? : string;
	rightChar?: string;

[twlite]

Suggested change

	separator? : string;
	separator?: string;

[Crocross]

I forgot all about changing progress bar stuff.
This makes sense to me, similar to my previous idea from Discord!

Thanks for doing this cause I completely forgot. I originally had the following:

• indicator
• length
• emptyLine
• filledLine
• timecodes
• timecodeSeparator

Which I think is all covered :)

[twlite]

LGTM!