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

Jump to bottom

ScrollView behaviour on test fabric app inconsistent with current ScrollView example on Gallery #12664

Closed
Tracked by #384
Yajur-Grover opened this issue Jan 30, 2024 · 2 comments
Closed
Tracked by #384

ScrollView behaviour on test fabric app inconsistent with current ScrollView example on Gallery #12664

Yajur-Grover opened this issue Jan 30, 2024 · 2 comments
Assignees
@Yajur-Grover
Labels
Area: Fabric Support Facebook Fabric Area: ScrollView New Architecture Broad category for issues that apply to the RN "new" architecture of Turbo Modules + Fabric
Milestone
Next

Comments

@Yajur-Grover
Copy link
Contributor

Yajur-Grover commented Jan 30, 2024 •
edited
Loading

When testing the ScrollView component example page on a Fabric app, there were several inconsitincies in behaviour with the current version of the example page on Gallery.

  • On the fabric app, I am unable to scroll through the 'Simple ScrollView with text' and 'Horizontal ScrollView' examples. On the 'Horizontal ScrollView' example, there is no scroll bar showing either.

  • The 'Disabled ScrollView' example is enabled on the fabric test app, and allows the user to scroll through the component.

  • In the 'Nested ScrollView with opposite scroll direction', the nested ScrollView does not respond to scrolling input.
@Yajur-Grover Yajur-Grover mentioned this issue Jan 30, 2024
Open
@microsoft-github-policy-service microsoft-github-policy-service bot added the Needs: Triage 🔍 New issue that needs to be reviewed by the issue management team (label applied by bot) label Jan 30, 2024
@Yajur-Grover Yajur-Grover changed the title Unable to scroll through ScrollView components on example page Unable to scroll through ScrollView components on example page except disabled ScrollView Jan 30, 2024
@Yajur-Grover Yajur-Grover changed the title Unable to scroll through ScrollView components on example page except disabled ScrollView ScrollView behaviour on test fabric app inconsistent with current ScrollView example on Gallery Jan 30, 2024
@chrisglein
Copy link
Member

@Yajur-Grover Which version is this on? 0.73.x, or canary, and if so, which canary version?
For example, scroll bars were not implemented until recently.

There are some not implemented things across the board here. Moving to main RNW repo.

@chrisglein chrisglein transferred this issue from microsoft/react-native-gallery Feb 1, 2024
@jonthysell jonthysell added Area: ScrollView Area: Fabric Support Facebook Fabric and removed Needs: Triage 🔍 New issue that needs to be reviewed by the issue management team (label applied by bot) labels Feb 5, 2024
@jonthysell jonthysell added this to the Next milestone Feb 5, 2024
@microsoft-github-policy-service microsoft-github-policy-service bot added the Invalid Triage https://github.com/microsoft/react-native-windows/wiki/Triage-Process (label applied by bot) label Feb 6, 2024
@Yajur-Grover Yajur-Grover mentioned this issue Feb 8, 2024
Merged
@Yajur-Grover
Copy link
Contributor Author

@Yajur-Grover Which version is this on? 0.73.x, or canary, and if so, which canary version? For example, scroll bars were not implemented until recently.

There are some not implemented things across the board here. Moving to main RNW repo.

Looks like pulling #12664 resolved this issue. The changes made should be included in the PR mentioned above.

@jonthysell jonthysell added the New Architecture Broad category for issues that apply to the RN "new" architecture of Turbo Modules + Fabric label Mar 7, 2024
@jonthysell jonthysell removed the Invalid Triage https://github.com/microsoft/react-native-windows/wiki/Triage-Process (label applied by bot) label Aug 28, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@Yajur-Grover Yajur-Grover

Labels
Area: Fabric Support Facebook Fabric Area: ScrollView New Architecture Broad category for issues that apply to the RN "new" architecture of Turbo Modules + Fabric
Projects
RNW New Architecture
Status: Done
Milestone
Next
Development

No branches or pull requests
3 participants
@jonthysell @chrisglein @Yajur-Grover