Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix progress bar rendering #949

Merged
merged 3 commits into from
May 19, 2020

Conversation

scholtzan
Copy link
Collaborator

Fixes #919

The problem causing this rendering issue is that insets get applied to a Rect with zero width. This results in a Rect of some negative width getting displayed when the progress bar is at zero.

let rounded_rect = Rect::from_origin_size(
Point::ORIGIN,
Point::new(-inset, 0.),
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

We need to translate the rect to respect the x inset.

@xStrom
Copy link
Member

xStrom commented May 17, 2020

Thanks for the fix!

Please also add an entry to CHANGELOG.md - more instructions here.

@xStrom xStrom added the S-waiting-on-author waits for changes from the submitter label May 17, 2020
scholtzan added a commit to scholtzan/druid that referenced this pull request May 17, 2020
@scholtzan scholtzan removed the S-waiting-on-author waits for changes from the submitter label May 17, 2020
@scholtzan
Copy link
Collaborator Author

Added an entry to the changelog

Copy link
Member

@xStrom xStrom left a comment

Choose a reason for hiding this comment

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

Thanks, this works!

PS. I added the missing link in the changelog. We're hoping to automate the changelog links soon.

scholtzan added a commit to scholtzan/druid that referenced this pull request May 17, 2020
@cmyr
Copy link
Member

cmyr commented May 18, 2020

Thanks @scholtzan!

@scholtzan scholtzan merged commit 3136d26 into linebender:master May 19, 2020
scholtzan added a commit that referenced this pull request May 19, 2020
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.

rendering issue with progress bar
3 participants