Skip to content

WIP feat(ios): add useLegacy flag - #763

Closed
troZee wants to merge 5 commits into
nextfrom
feat/add-fallback-component
Closed

WIP feat(ios): add useLegacy flag#763
troZee wants to merge 5 commits into
nextfrom
feat/add-fallback-component

Conversation

@troZee

@troZeetroZee commented Aug 31, 2023

Copy link
Copy Markdown
Collaborator

Summary

  • added useLegacy flag as a fallback to a legacy component (UIPageViewController)
  • Changed name to RNCScrollViewPager to don't introduce any breaking changes for expo go
  • Created the legacy dir in the ios folder

Test Plan

  • Run the example
  • Click on Legacy example
  • Do some testing
  • Click on Basic example
  • Do some testing

Compatibility

OSImplemented
iOS
Android

Checklist

  • I have tested this on a device and a simulator
  • I added the documentation in README.md
  • I updated the typed files (TS and Flow)
  • check new arch
  • fix Android
  • fix pipeline

@troZee

Copy link
Copy Markdown
CollaboratorAuthor

Screenshot_1693489802

okwasniewski
okwasniewski previously approved these changes Aug 31, 2023
@@ -0,0 +1,8 @@
<?xml version="1.0" encoding="UTF-8"?>

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

This can be removed

pageMargin={10}
// Lib does not support dynamically orientation change
orientation="horizontal"
useLegacy={true}

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

You can specify just useLegacy without passing true

@troZee

Copy link
Copy Markdown
CollaboratorAuthor

Screenshot_1693505469
Screenshot_1693505510

Spacing issue

@troZeetroZee mentioned this pull request Aug 31, 2023
@igorbejigorbej self-assigned this Sep 12, 2023
@troZeetroZee changed the title feat(ios): add useLegacy flagWIP feat(ios): add useLegacy flagSep 25, 2023
@troZee
troZee marked this pull request as draft October 24, 2023 12:34
@troZee

Copy link
Copy Markdown
CollaboratorAuthor

Closed in favor of #783

@troZeetroZee closed this Nov 24, 2023
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@troZee@okwasniewski@igorbej