Skip to content

Conversation

troZee
Copy link
Collaborator

@troZee troZee commented Mar 29, 2024

Summary

Test Plan

What's required for testing (prerequisites)?

What are the steps to reproduce (after prerequisites)?

Compatibility

OS Implemented
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)

krozniata and others added 30 commits January 17, 2023 17:57
* feat(iOS): change Fabric implementation to UIScrollView

* fix: fix offset values in vertical orientation

* feat: add initialPage props support

* feat: add RTL language support

* feat: add pageMargin prop support

* fix: fix typescript error

* feat: remove React.cloneElement

* feat(ios): add getPageOffset method

* fix: fix styles in old example

* fix: behavior on page remove
Co-authored-by: Piotr Trocki <[email protected]>
* feat: rewrite old arch to use UIScrollView

* feat: update example styles

* fix: sending event on scrollViewDidEndDecelerating

* feat: properly calculate width using orientation

* fix: change way of disabing scroll

* feat: rename to RNCPagerView

* fix: removing last page

* fix: remove unused properties, set animated
…tion-swipe

fix(ios): v7 swipe to go back gestures with react navigation
fix: multiple updates of frame and contentSize
v7.0.0 feat: add button to quickly switch layout direction
Co-authored-by: Piotr Trocki <[email protected]>
chore(next): upgrade RN version in both examples
…mplementation (#783)

* wip: unsuspicious changes

* wip: unsuspicious changes v2

* FABRIC NEW IMPL -> OLD IMPL

* wip: bring back useLegacy on the RN side

* wip: bring back duplicate types to fix codegen issues

* wip: remove #705 related code for now

* wip: old/new impl division first draft

* wip: old/new impl division continued

* wip: old/new impl v3

* wip: add a `LEGACY_` prefix to all legacy implementation-related symbols

* wip: fix styles for new implementation on Fabric

* wip: move old/new impl into separate folders

* wip: fix old impl fabric symbol names

* wip: xcode changes

* wip: clean up & unify the naming convention

* wip: fix linter issues

* wip: fix styles for new implementation on Paper

* wip: make Fabric example run on another port by default to make it possible to run both examples in parallel

* wip: implement an abstraction over native commands invocations to reduce branching

* refactor: remove the unnecessary value for boolean props

* fix: bump react-native-safe-area-context to a Fabric-enabled version

* feat: bring back & adjust the `bootstrap-fabric` script

* feat: adjust the home screen title depending on the used architecture

* chore: update example/Podfile.lock

* chore: update an Xcode project file after building

* ci: make next branch events trigger ios/android build workflows

* chore: remove commented-out code related to #705 for now

* chore: add legacy implementation explanation comment

* wip: Android fixes

* fix: unnecessary comma in MainActivity.java

* feat: readme makeover

* chore: bump react-native-screens & react-native-gesture-handler in example

* refactor(android): extract module name to shared variable, add comment for context

* chore: remove unnecessary yarn.lock deps

* chore(ios): bring back removed build flags

* chore(ios): remove unnecessary concurrentRootEnabled method

According to React Native Upgrade Helper, this method is to be removed when updating to RN 0.72:
https://react-native-community.github.io/upgrade-helper/?from=0.71.14&to=0.72.0#RnDiffApp-ios-RnDiffApp-AppDelegate.mm

* fix(android): adjust incorrect param type on Fabric

* chore: remove unnecessary tsconfig.json comment

* chore(ios): bring back (currently unused) code related to #712 and #705
Piotr Trocki added 4 commits March 29, 2024 11:41
# Conflicts:
#	.github/workflows/android.yml
#	.github/workflows/main.yml
#	.gitignore
#	README.md
#	example/Gemfile
#	example/Gemfile.lock
#	example/android/app/build.gradle
#	example/android/app/src/main/AndroidManifest.xml
#	example/android/app/src/main/java/com/pagerviewexample/MainActivity.java
#	example/android/build.gradle
#	example/android/gradlew.bat
#	example/ios/PagerViewExample.xcworkspace/xcshareddata/IDEWorkspaceChecks.plist
#	example/ios/Podfile
#	example/ios/Podfile.lock
#	example/metro.config.js
#	example/package.json
#	example/src/App.tsx
#	example/src/LegacyBasicPagerViewExample.tsx
#	example/yarn.lock
#	ios/Fabric/RNCPagerViewComponentView.mm
#	ios/LEGACY/LEGACY_RNCPagerView.m
#	maestro-tests/basic_example/tc14_pager_basic_example.yaml.yaml
#	package.json
#	react-native-pager-view.podspec
#	yarn.lock
Sign up for free to 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.

6 participants