juliaup icon indicating copy to clipboard operation
juliaup copied to clipboard

Update progress bar style to match 1.12 progress bar

Open christiangnrd opened this issue 1 year ago • 8 comments

Couldn't get the smoother version without causing some issues with the colour printing.

I moved the ':' out of the prefix as it was too blue for me but I can revert that if people don't like it.

christiangnrd avatar Oct 08 '24 17:10 christiangnrd

It would be good to use both the left and right half-bar characters.

tecosaur avatar Oct 09 '24 05:10 tecosaur

It would be good to use both the left and right half-bar characters.

@tecosaur I agree, but I couldn't find how to have the right half-bar character use the alternate colour instead of the main colour.

image

If there's an easy fix suggested, I'll integrate it with this PR. Otherwise, I suggest we merge this until a more involved solution is implemented.

christiangnrd avatar Oct 09 '24 15:10 christiangnrd

Didn't mean to close sorry for the spam

christiangnrd avatar Oct 09 '24 15:10 christiangnrd

We're using a progress bar library? The only non-trivial component in implementing a progress bar ourselves is the ETA.

tecosaur avatar Oct 09 '24 15:10 tecosaur

IMO this should be merged as-is for now as it's still an improvement, and I'll open an issue to improve it so that I can remember to return to it in an unknown amount of time if someone else hasn't gotten to it first.

christiangnrd avatar Oct 12 '24 01:10 christiangnrd

Hm, I have to admit I find that visually way more difficult to distinguish than the existing progress bar... Everything is super small (why is this like two pixels high, only?), the difference in color between "finished" and "in progress" (blue and black) is hard to see... All the default options shown here https://crates.io/crates/indicatif (the progress bar package we use) are way easier on the eye to me...

davidanthoff avatar Oct 12 '24 06:10 davidanthoff

Even if it's not visually great to some eyes (I suspect this is a little font-dependent too), I do think there's a fair bit of value in consistency with Pkg.

tecosaur avatar Oct 12 '24 09:10 tecosaur

Let's hold this for now, the earliest we should merge this is when 1.12 is actually released.

davidanthoff avatar Oct 12 '24 18:10 davidanthoff

I think the bar can be smoothened a bit similar to how it was done in https://github.com/JuliaLang/Pkg.jl/pull/4038.

KristofferC avatar Oct 20 '25 20:10 KristofferC

Did that here https://github.com/JuliaLang/juliaup/pull/1287

IanButterworth avatar Oct 21 '25 14:10 IanButterworth

Happy to add the commit here if you agree.

IanButterworth avatar Oct 21 '25 14:10 IanButterworth

Thank you! I don't know when I would have had time to smooth it out

Happy to add the commit here if you agree.

#1287 includes my commits plus yours so I'll just close this one and the code can be finalized over there

christiangnrd avatar Oct 21 '25 14:10 christiangnrd