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.

Sign up for GitHub

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

#23546 - Wrong Image opens on HT Report in Safari #28441

Closed
wants to merge 8 commits into from
Closed

#23546 - Wrong Image opens on HT Report in Safari #28441

Changes from 1 commit
Commits
Show all changes
8 commits
Select commit Hold shift + click to select a range
35ae16a
#23546 - Wrong Image opens on HT Report in Safari
mikesmnx Sep 27, 2023
3d4faa6
pagingEnabled removed
mikesmnx Sep 29, 2023
d1bf919
Updated approach
mikesmnx Oct 12, 2023
afcac5d
Merge remote-tracking branch 'upstream/main' into mikesmnx-issue-23546
mikesmnx Oct 23, 2023
d10f9fd
Fixed lint error
mikesmnx Nov 2, 2023
6f41ef5
Prettier fix
mikesmnx Nov 2, 2023
60ac519
Merge remote-tracking branch 'upstream/main' into mikesmnx-issue-23546
mikesmnx Dec 8, 2023
5a19e1f
Added comment on contentInset
mikesmnx Dec 9, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
5 changes: 3 additions & 2 deletions src/components/Attachments/AttachmentCarousel/index.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -25,6 +25,7 @@ const viewabilityConfig = {
// To facilitate paging through the attachments, we want to consider an item "viewable" when it is
// more than 95% visible. When that happens we update the page index in the state.
itemVisiblePercentThreshold: 95,
minimumViewTime: 300,
};

function AttachmentCarousel({report, reportActions, source, onNavigate, setDownloadButtonVisibility, translate}) {
Expand Down Expand Up @@ -189,7 +190,7 @@ function AttachmentCarousel({report, reportActions, source, onNavigate, setDownl
bounces={false}
// Scroll only one image at a time no matter how fast the user swipes
disableIntervalMomentum
pagingEnabled
pagingEnabled={false}
Copy link
Contributor

Choose a reason for hiding this comment

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

can't you just remove it?

Copy link
Author

Choose a reason for hiding this comment

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

@nkuoch done

snapToAlignment="start"
snapToInterval={containerWidth}
// Enable scrolling by swiping on mobile (touch) devices only
Expand All @@ -198,7 +199,7 @@ function AttachmentCarousel({report, reportActions, source, onNavigate, setDownl
scrollEnabled={canUseTouchScreen}
ref={scrollRef}
initialScrollIndex={page}
initialNumToRender={3}
initialNumToRender={1}
windowSize={5}
maxToRenderPerBatch={3}
data={attachments}
Expand Down