Skip to content

Support for unicode characters and a right to left progress bar#86

Merged
gaborcsardi merged 1 commit into
r-lib:masterfrom
jimhester:unicode-and-reverse
Feb 15, 2019
Merged

Support for unicode characters and a right to left progress bar#86
gaborcsardi merged 1 commit into
r-lib:masterfrom
jimhester:unicode-and-reverse

Conversation

@jimhester

Copy link
Copy Markdown
Member

These were the changes needed for my extremely important vroom progress bar (https://asciinema.org/a/FSsKkqqxmTQ6vjI8DWy2LqbRp)

@jimhester

Copy link
Copy Markdown
Member Author

I am not entirely sure I will keep this PB in vroom, but I think this could still be useful in general.

@gaborcsardi

Copy link
Copy Markdown
Member

Thanks, looks good!

@gaborcsardi gaborcsardi merged commit 2ba1a03 into r-lib:master Feb 15, 2019
Comment thread inst/include/RProgress.h
double start; // Start time
bool toupdate; // Are we updating? (After show_after.)
bool complete; // Are we complete?
bool reverse; // go from right to left rather than left to right

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

I think I neglected to initialize this in the ctor above

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants