Page MenuHomePhabricator

[native] Fix incorrectly set verticalBounds in UserProfileAvatar
ClosedPublic

Authored by ashoat on Nov 4 2024, 1:54 PM.
Tags
None
Referenced Files
Unknown Object (File)
Mon, Dec 16, 9:27 PM
Unknown Object (File)
Mon, Dec 16, 9:27 PM
Unknown Object (File)
Mon, Dec 16, 9:26 PM
Unknown Object (File)
Mon, Dec 16, 9:26 PM
Unknown Object (File)
Wed, Dec 4, 11:18 AM
Unknown Object (File)
Fri, Nov 29, 8:12 AM
Unknown Object (File)
Thu, Nov 28, 2:58 PM
Unknown Object (File)
Thu, Nov 28, 12:55 PM
Subscribers

Details

Summary

verticalBounds of FullScreenViewModal are using to establish a border that will "cut off" part of the image as it animates back.

In the chat view, we use to prevent the animating image from appearing over the tab bar and the header.

In his initial implementation of UserProfileAvatar, Ginsu seems to have set verticalBounds based on an uninformed guess as to its function.

We don't have any requirements for the profile bottom sheet to have a border that cuts off the image as it animates back, since the bottom sheet appears above (over) navigational elements such as the tab bar and the header.

As such, we can effectively disable verticalBounds by setting it to be the full screen.

This fixes a visual issue where part of the user avatar gets cut off when animating back.

Depends on D13868

Test Plan

I confirmed the visual glitch was gone

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable