-
Notifications
You must be signed in to change notification settings - Fork 446
feat(iOS): Add a useLegacy
flag to switch between the old/new iOS implementation
#783
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
Merged
Merged
Changes from 37 commits
Commits
Show all changes
40 commits
Select commit
Hold shift + click to select a range
686023c
wip: unsuspicious changes
igorbej 7a2695c
wip: unsuspicious changes v2
igorbej faed701
FABRIC NEW IMPL -> OLD IMPL
igorbej b7ac223
wip: bring back useLegacy on the RN side
igorbej fa06f8b
wip: bring back duplicate types to fix codegen issues
igorbej 8369060
wip: remove #705 related code for now
igorbej 036e286
wip: old/new impl division first draft
igorbej 13b05df
wip: old/new impl division continued
igorbej 1b9dbd0
wip: old/new impl v3
igorbej 4e707ce
wip: add a `LEGACY_` prefix to all legacy implementation-related symbols
igorbej 7586f13
wip: fix styles for new implementation on Fabric
igorbej 5746c99
wip: move old/new impl into separate folders
igorbej e80ad6c
wip: fix old impl fabric symbol names
igorbej f293140
wip: xcode changes
igorbej 238afe7
wip: clean up & unify the naming convention
igorbej 5e2af1c
wip: fix linter issues
igorbej e90d004
wip: fix styles for new implementation on Paper
igorbej b669e98
wip: make Fabric example run on another port by default to make it po…
igorbej b56e124
wip: implement an abstraction over native commands invocations to red…
igorbej f579d11
refactor: remove the unnecessary value for boolean props
igorbej b73f656
Merge commit '3ea6dae21d3a2f78233720f66343034ccba990f0' into add-useL…
igorbej 67e3ec6
fix: bump react-native-safe-area-context to a Fabric-enabled version
igorbej a6f6bcf
feat: bring back & adjust the `bootstrap-fabric` script
igorbej 8e676b5
feat: adjust the home screen title depending on the used architecture
igorbej 7aeebcd
chore: update example/Podfile.lock
igorbej bc51f9b
chore: update an Xcode project file after building
igorbej 6e3507e
ci: make next branch events trigger ios/android build workflows
igorbej 0a3db4f
chore: remove commented-out code related to #705 for now
igorbej ed08c33
chore: add legacy implementation explanation comment
igorbej 384cde4
wip: Android fixes
igorbej e2d5ae9
fix: unnecessary comma in MainActivity.java
igorbej 99412a7
feat: readme makeover
igorbej 19785bb
chore: bump react-native-screens & react-native-gesture-handler in ex…
igorbej 6a54d0a
refactor(android): extract module name to shared variable, add commen…
igorbej 2fbccf2
chore: remove unnecessary yarn.lock deps
igorbej 0392c94
chore(ios): bring back removed build flags
igorbej c7139c1
chore(ios): remove unnecessary concurrentRootEnabled method
igorbej 2a97ac7
fix(android): adjust incorrect param type on Fabric
igorbej 0dd9b2a
chore: remove unnecessary tsconfig.json comment
igorbej 0212f97
chore(ios): bring back (currently unused) code related to #712 and #705
igorbej File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
17 changes: 17 additions & 0 deletions
17
android/src/fabric/java/com/reactnativepagerview/LEGACY_PagerViewViewManager.kt
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,17 @@ | ||
package com.reactnativepagerview | ||
|
||
import com.facebook.react.uimanager.ViewGroupManager | ||
import android.widget.FrameLayout | ||
import com.facebook.react.module.annotations.ReactModule | ||
import com.facebook.react.uimanager.ThemedReactContext | ||
|
||
// Note: The LEGACY_ variant is an iOS-only feature and the related Android files | ||
// are only included because of and relevant to auxiliary processes, like Codegen. | ||
@ReactModule(name = LEGACY_PagerViewViewManagerImpl.NAME) | ||
class LEGACY_PagerViewViewManager : ViewGroupManager<FrameLayout>() { | ||
override fun getName() = LEGACY_PagerViewViewManagerImpl.NAME | ||
|
||
override fun createViewInstance(context: ThemedReactContext): FrameLayout { | ||
throw Error("LEGACY_RNCViewPager is an iOS-only feature") | ||
} | ||
} |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
7 changes: 7 additions & 0 deletions
7
android/src/main/java/com/reactnativepagerview/LEGACY_PagerViewViewManagerImpl.kt
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,7 @@ | ||
package com.reactnativepagerview | ||
|
||
// Note: The LEGACY_ variant is an iOS-only feature and the related Android files | ||
// are only included because of and relevant to auxiliary processes, like Codegen. | ||
object LEGACY_PagerViewViewManagerImpl { | ||
const val NAME = "LEGACY_RNCViewPager" | ||
} |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
17 changes: 17 additions & 0 deletions
17
android/src/paper/java/com/reactnativepagerview/LEGACY_PagerViewViewManager.kt
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,17 @@ | ||
package com.reactnativepagerview | ||
|
||
import com.facebook.react.uimanager.ViewGroupManager | ||
import android.widget.FrameLayout | ||
import com.facebook.react.module.annotations.ReactModule | ||
import com.facebook.react.uimanager.ThemedReactContext | ||
|
||
// Note: The LEGACY_ variant is an iOS-only feature and the related Android files | ||
// are only included because of and relevant to auxiliary processes, like Codegen. | ||
@ReactModule(name = LEGACY_PagerViewViewManagerImpl.NAME) | ||
class LEGACY_PagerViewViewManager : ViewGroupManager<FrameLayout>() { | ||
override fun getName() = LEGACY_PagerViewViewManagerImpl.NAME | ||
|
||
override fun createViewInstance(context: ThemedReactContext): FrameLayout { | ||
throw Error("LEGACY_RNCViewPager is an iOS-only feature") | ||
} | ||
} |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
8 changes: 8 additions & 0 deletions
8
example/ios/PagerViewExample.xcworkspace/xcshareddata/IDEWorkspaceChecks.plist
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,8 @@ | ||
<?xml version="1.0" encoding="UTF-8"?> | ||
troZee marked this conversation as resolved.
Show resolved
Hide resolved
|
||
<!DOCTYPE plist PUBLIC "-//Apple//DTD PLIST 1.0//EN" "http://www.apple.com/DTDs/PropertyList-1.0.dtd"> | ||
<plist version="1.0"> | ||
<dict> | ||
<key>IDEDidComputeMac32BitWarning</key> | ||
<true/> | ||
</dict> | ||
</plist> |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.