-
Notifications
You must be signed in to change notification settings - Fork 1.5k
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Scroll to focused media below header
Summary: ## Context We want the post-tap headers to be sticky so that context is maintained as the user scrolls down the media chain. ## In this diff When clicking on a media tile in a media module (rather than the See All CTA), the sticky header was covering part of the media post's header. We want to shift the scroll-to view down so that the sticky header is sitting on top of the media post's header. - Create new version of IGListAdapter API's `scrollToObject` function that contains an additional parameter `additionalYOffset` - Call new `scrollToObject` function with offset of 48.0f (height of header) ## In this stack Add sticky headers for post-taps, excluding SERP Reviewed By: maxolls Differential Revision: D29961626 fbshipit-source-id: c8644cc868e8d7ec92cad5f6c7742d93d001f67d
- Loading branch information
1 parent
936f558
commit f2166c3
Showing
4 changed files
with
75 additions
and
47 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.