Skip to content
This repository was archived by the owner on Aug 1, 2023. It is now read-only.

Draw ShimmerDrawable mDrawRect at correct position#60

Closed
yarolegovich wants to merge 1 commit intofacebookarchive:masterfrom
yarolegovich:59-shimmer-position
Closed

Draw ShimmerDrawable mDrawRect at correct position#60
yarolegovich wants to merge 1 commit intofacebookarchive:masterfrom
yarolegovich:59-shimmer-position

Conversation

@yarolegovich
Copy link
Copy Markdown
Contributor

After translation, rotation pivot (width/2, height/2) doesn't correspond to the center of mDrawRect, which leads to an incorrect mDrawRect position.
Perform translation after rotation to fix the issue.

re #59

After translation, rotation pivot (width/2, height/2) doesn't correspond to the center of mDrawRect, which leads to an incorrect mDrawRect position.
Perform translation after rotation to fix the issue.

re #59
Copy link
Copy Markdown

@facebook-github-bot facebook-github-bot left a comment

Choose a reason for hiding this comment

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

@xiphirx is landing this pull request. If you are a Facebook employee, you can view this diff on Phabricator.

PoppyProses added a commit to PoppyProses/PoppyProses that referenced this pull request Aug 20, 2025
Summary:
After translation, rotation pivot (width/2, height/2) doesn't correspond to the center of mDrawRect, which leads to an incorrect mDrawRect position.
Perform translation after rotation to fix the issue.

re #59
Closes facebookarchive/shimmer-android#60

Differential Revision: D8728413

Pulled By: xiphirx

fbshipit-source-id: b8b839fae2e9c91b4a5085e020460447bd6f7cad
FelixiReed added a commit to FelixiReed/monoap that referenced this pull request Sep 17, 2025
Summary:
After translation, rotation pivot (width/2, height/2) doesn't correspond to the center of mDrawRect, which leads to an incorrect mDrawRect position.
Perform translation after rotation to fix the issue.

re #59
Closes facebookarchive/shimmer-android#60

Differential Revision: D8728413

Pulled By: xiphirx

fbshipit-source-id: b8b839fae2e9c91b4a5085e020460447bd6f7cad
overbomigu added a commit to overbomigu/imgur that referenced this pull request Nov 22, 2025
Summary:
After translation, rotation pivot (width/2, height/2) doesn't correspond to the center of mDrawRect, which leads to an incorrect mDrawRect position.
Perform translation after rotation to fix the issue.

re #59
Closes facebookarchive/shimmer-android#60

Differential Revision: D8728413

Pulled By: xiphirx

fbshipit-source-id: b8b839fae2e9c91b4a5085e020460447bd6f7cad
AstarLighte added a commit to AstarLighte/mixdown that referenced this pull request Dec 16, 2025
Summary:
After translation, rotation pivot (width/2, height/2) doesn't correspond to the center of mDrawRect, which leads to an incorrect mDrawRect position.
Perform translation after rotation to fix the issue.

re #59
Closes facebookarchive/shimmer-android#60

Differential Revision: D8728413

Pulled By: xiphirx

fbshipit-source-id: b8b839fae2e9c91b4a5085e020460447bd6f7cad
edsoftsae added a commit to edsoftsae/edsoftsae that referenced this pull request Jan 4, 2026
Summary:
After translation, rotation pivot (width/2, height/2) doesn't correspond to the center of mDrawRect, which leads to an incorrect mDrawRect position.
Perform translation after rotation to fix the issue.

re #59
Closes facebookarchive/shimmer-android#60

Differential Revision: D8728413

Pulled By: xiphirx

fbshipit-source-id: b8b839fae2e9c91b4a5085e020460447bd6f7cad
owpdoeir added a commit to owpdoeir/mmap_ringbuffer that referenced this pull request Mar 27, 2026
Summary:
After translation, rotation pivot (width/2, height/2) doesn't correspond to the center of mDrawRect, which leads to an incorrect mDrawRect position.
Perform translation after rotation to fix the issue.

re #59
Closes facebookarchive/shimmer-android#60

Differential Revision: D8728413

Pulled By: xiphirx

fbshipit-source-id: b8b839fae2e9c91b4a5085e020460447bd6f7cad
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants