Question: Why default to full screen height instead of allowing dynamic height?

Which OS ?

Both

Version

Which versions are you using:

• react-native-swiper v1.6.0.rc
• react-native v0.62.2

Problem / Question

In the implementation, it is setting the initial height state to window height if we do not provide a height prop to it.

In order to place the swiper perfectly, I would need to calculate the height and width of the content beforehand which is kind of unnecessary work.

Wouldn’t it be more flexible to not do anything with the initial height and let the container fit dynamically with the content? We can always control the dimension ourselves.

PS. If there’s a way to have dynamic height control already that I missed, please let me know thanks!

[cardiohelge]

I build a workaround by using onLayout in the child views: (The largest child defines the height of the Swiper)

const [height, setHeight] = useState(100); // "minHeight" - can be smaller too

const handleSetHeight = (value) => {
  if (value > height) {
    setHeight(value);
  }
};

return (
  <Swiper ... style={{ height }}>
    {children.map((child) => (<View onLayout={(event) =>handleSetHeight(event.nativeEvent.layout.height)}>{child}</View>))}
  </Swiper>
);

[danielfx90]

Anyone to answer this, please? It’s a great package, but setting hardcoded dimensions complicates everything.