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.

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

fix: [Horizontal Scroll previous button doesn't work in 2 column width screen size] in [Horizontal Scroll] #5676

Closed
JennyzhaoM opened this issue Mar 1, 2022 · 0 comments · Fixed by #5680
Assignees
Labels
area:fast-foundation Pertains to fast-foundation bug A bug status:needs-investigation Needs additional investigation

Comments

@JennyzhaoM
Copy link

JennyzhaoM commented Mar 1, 2022

🐛 Bug Report

💻 Repro or Code Sample

Change screen width to makes the Default scroll show only "Card number 1" and "Card number 2".
Click the ">" button to scroll left,
Click the "< " button to scroll right
Observe it is not able to scroll back to previous card 1, it stays at the current position.

🤔 Expected Behavior

Scroll back to the "card number 1" position, and display both "card #1" and "card #2"

😯 Current Behavior

It is not able to scroll back to previous card 1, it stays at the current position.

💁 Possible Solution

handle the case when nextIndex equal to current

🔦 Context

🌍 Your Environment

Microsoft Edge

  • OS & Device: Windows on PC
  • Browser Microsoft Edge
  • Version 98.0.1108.62 (Official build) (64-bit), Window 11
@JennyzhaoM JennyzhaoM added the status:triage New Issue - needs triage label Mar 1, 2022
@EisenbergEffect EisenbergEffect added area:fast-components area:fast-foundation Pertains to fast-foundation bug A bug status:needs-investigation Needs additional investigation labels Mar 1, 2022
@EisenbergEffect EisenbergEffect removed the status:triage New Issue - needs triage label Mar 1, 2022
@robarbms robarbms self-assigned this Mar 1, 2022
@robarbms robarbms linked a pull request Mar 1, 2022 that will close this issue
9 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area:fast-foundation Pertains to fast-foundation bug A bug status:needs-investigation Needs additional investigation
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants