Skip to content

Fix progress bit overflow bug#110411

Merged
bpasero merged 2 commits intomicrosoft:masterfrom
YuJianghao:patch-1
Nov 12, 2020
Merged

Fix progress bit overflow bug#110411
bpasero merged 2 commits intomicrosoft:masterfrom
YuJianghao:patch-1

Conversation

@YuJianghao
Copy link
Contributor

parent width: 5000%
progress bit width: 100%
translateX should be:

  50%: 5000% * 50% = 2500%
 100%: 5000% * 100% - 100% (do not overflow): 4900%

translateX(4950%) will make half of progress bit overflow. :p

bit: 100% width, parent 5000% width, translateX(4900%) + bitwidth(100%) = parentwidth(5000%). 4950% will make half of progress bit overflow. :p
Copy link

@rheh rheh left a comment

Choose a reason for hiding this comment

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

LGMT

@bpasero bpasero added this to the November 2020 milestone Nov 12, 2020
@bpasero
Copy link
Member

bpasero commented Nov 12, 2020

Thanks 👍

@bpasero bpasero merged commit fb0128c into microsoft:master Nov 12, 2020
meganrogge pushed a commit that referenced this pull request Nov 18, 2020
* Fix progress bit overflow bug

bit: 100% width, parent 5000% width, translateX(4900%) + bitwidth(100%) = parentwidth(5000%). 4950% will make half of progress bit overflow. :p

* Set progress bit to center at 50% animate
@github-actions github-actions bot locked and limited conversation to collaborators Dec 27, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants