Skip to content

Conversation

@FroggieFrog
Copy link
Contributor

@FroggieFrog FroggieFrog commented Aug 2, 2025

fixes #1872

This PR updates the available Spinners according to https://github.com/sindresorhus/cli-spinners.
As part of the update there are a bunch of new Spinners available:

  • Dots13
  • Dots14
  • DotsCircle
  • Binary
  • FingerDance
  • FistBump
  • SoccerHeader
  • Mindblown
  • Speaker
  • OrangePulse
  • BluePulse
  • OrangeBluePulse
  • TimeTravel
  • DwarfFortress

spinner

  • I have read the Contribution Guidelines
  • I have commented on the issue above and discussed the intended changes
  • A maintainer has signed off on the changes and the issue was assigned to me
  • All newly added code is adequately covered by tests
  • All existing tests are still running without errors
  • The documentation was modified to reflect the changes OR no documentation changes are required.

Changes

used Testcode
        Console.OutputEncoding = System.Text.Encoding.Unicode;

        var delay = 3000;
        var message = "Thinking ...";

        await AnsiConsole.Status()
            .Spinner(Spinner.Known.Dots13)
            .StartAsync($"[bold red]{nameof(Spinner.Known.Dots13)}[/] {message}", async ctx =>
            {
                await Task.Delay(delay);

                ctx.Spinner = Spinner.Known.Dots14;
                ctx.Status = $"[bold red]{nameof(Spinner.Known.Dots14)}[/] {message}";
                await Task.Delay(delay);

                ctx.Spinner = Spinner.Known.DotsCircle;
                ctx.Status = $"[bold red]{nameof(Spinner.Known.DotsCircle)}[/] {message}";
                await Task.Delay(delay);

                ctx.Spinner = Spinner.Known.Binary;
                ctx.Status = $"[bold red]{nameof(Spinner.Known.Binary)}[/] {message}";
                await Task.Delay(delay);

                ctx.Spinner = Spinner.Known.FingerDance;
                ctx.Status = $"[bold red]{nameof(Spinner.Known.FingerDance)}[/] {message}";
                await Task.Delay(delay);

                ctx.Spinner = Spinner.Known.FistBump;
                ctx.Status = $"[bold red]{nameof(Spinner.Known.FistBump)}[/] {message}";
                await Task.Delay(delay);

                ctx.Spinner = Spinner.Known.SoccerHeader;
                ctx.Status = $"[bold red]{nameof(Spinner.Known.SoccerHeader)}[/] {message}";
                await Task.Delay(delay * 2);

                ctx.Spinner = Spinner.Known.Mindblown;
                ctx.Status = $"[bold red]{nameof(Spinner.Known.Mindblown)}[/] {message}";
                await Task.Delay(delay * 2);

                ctx.Spinner = Spinner.Known.Speaker;
                ctx.Status = $"[bold red]{nameof(Spinner.Known.Speaker)}[/] {message}";
                await Task.Delay(delay);

                ctx.Spinner = Spinner.Known.OrangePulse;
                ctx.Status = $"[bold red]{nameof(Spinner.Known.OrangePulse)}[/] {message}";
                await Task.Delay(delay);

                ctx.Spinner = Spinner.Known.BluePulse;
                ctx.Status = $"[bold red]{nameof(Spinner.Known.BluePulse)}[/] {message}";
                await Task.Delay(delay);

                ctx.Spinner = Spinner.Known.OrangeBluePulse;
                ctx.Status = $"[bold red]{nameof(Spinner.Known.OrangeBluePulse)}[/] {message}";
                await Task.Delay(delay);

                ctx.Spinner = Spinner.Known.TimeTravel;
                ctx.Status = $"[bold red]{nameof(Spinner.Known.TimeTravel)}[/] {message}";
                await Task.Delay(delay);

                ctx.Spinner = Spinner.Known.DwarfFortress;
                ctx.Status = $"[bold red]{nameof(Spinner.Known.DwarfFortress)}[/] {message}";
                await Task.Delay(delay * 5);
            });

Please upvote 👍 this pull request if you are interested in it.

@FroggieFrog
Copy link
Contributor Author

@microsoft-github-policy-service agree

@FroggieFrog FroggieFrog marked this pull request as ready for review August 2, 2025 18:30
Copy link
Contributor

@patriksvensson patriksvensson left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 👍

@patriksvensson
Copy link
Contributor

@FroggieFrog The build error is unrelated to your changes. I will fix that and notify you once it's done.

@github-actions github-actions bot added the ⭐ top pull request Top pull request. label Aug 3, 2025
@github-actions github-actions bot mentioned this pull request Aug 3, 2025
@patriksvensson patriksvensson merged commit 0889c2f into spectreconsole:main Aug 3, 2025
3 checks passed
@patriksvensson
Copy link
Contributor

Merged! Thank you for your contribution. Much appreciated! 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

⭐ top pull request Top pull request.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add missing Spinner dotsCircle

2 participants