react-native-pager-view icon indicating copy to clipboard operation
react-native-pager-view copied to clipboard

[6.3.1] next merged into master

Open Bardiamist opened this issue 1 year ago • 5 comments

6.3.1 problems (actual for 6.3.3)

In this commit https://github.com/callstack/react-native-pager-view/commit/280c3a3f86fa686c1c1a29c1c8499cf4d03c4231 next branch merged into master branch. Looks as mistake.

  1. Now TypeScript required useNext. What to use if I want stable version?

  2. PagerView stopped to pass props into pages on Android.

In 6.3.0 it was:

{
  "style": [
    null,
    {
      "position": "absolute",
      "left": 0,
      "right": 0,
      "top": 0,
      "bottom": 0
    }
  ]
}

Pages on iOS stil receive these props in 6.3.3. Also I remember here was collapsable: false. Should be missed even before 6.3.1. So what to do now, manually set collapsable={false} and style flex: 1?

Bardiamist avatar Jun 21 '24 07:06 Bardiamist

In this commit https://github.com/callstack/react-native-pager-view/commit/280c3a3f86fa686c1c1a29c1c8499cf4d03c4231 next branch merged into master branch. Looks as mistake.In this commit https://github.com/callstack/react-native-pager-view/commit/280c3a3f86fa686c1c1a29c1c8499cf4d03c4231 next branch merged into master branch. Looks as mistake.In this commit https://github.com/callstack/react-native-pager-view/commit/280c3a3f86fa686c1c1a29c1c8499cf4d03c4231 next branch merged into master branch. Looks as mistake.

Next is an optional implementation which usage is determined by useNext value, if you want to continue using stable implementation just dont add useNext prop.

PagerView stopped to pass props into pages on Android. Also I remember here was collapsable: false. Should be missed even before 6.3.1. So what to do now, manually set collapsable={false} and style flex: 1?

Please reproduce it in the example app

MrRefactor avatar Jun 26 '24 08:06 MrRefactor

if you want to continue using stable implementation just dont add useNext prop.

It's not optional prop in TypeScript

Bardiamist avatar Jun 26 '24 08:06 Bardiamist

@MrRefactor https://github.com/callstack/react-native-pager-view/issues/823#issuecomment-2124057082

Bardiamist avatar Jun 26 '24 08:06 Bardiamist

@MrRefactor Reproduced styles problem here https://github.com/Bardiamist/diff/tree/pager

Bardiamist avatar Jun 26 '24 08:06 Bardiamist

This should be an optional type, but in the mean time you can remove the TS error and maintain original behaviour by using useNext={false}

valtism avatar Jul 08 '24 05:07 valtism

Closing as per: https://github.com/callstack/react-native-pager-view/releases/tag/v6.6.0

MrRefactor avatar Nov 29 '24 10:11 MrRefactor