Scope store and work with SwiftUIX got wrong UI size

[marcuswu0814]

I'm trying to use the PaginationView provide by SwiftUIX library. But when I using the ForEach(store.scope) ... in the content builder, I got wrong size. It seems used a different initializer overload, and the page be considered a single view:

If I just using normal ForEach(store.pages)..., it work but it's not what I want.

The other initializer:

---

The reproduce project and detail description in here. If anyone has encountered a similar problem, please give me some advice. 🙏

---

I also create a issue at SwiftUIX.

[mbrandonw]

The issue seems to be in SwiftUIX. If you add @ViewBuilder to the content closure in the initializer you want, then it will work. You can either PR that to SwiftUIX, or you can drop this in your project:

extension PaginationView {
  public init<Data, ID>(
    axis: Axis = .horizontal,
    transitionStyle: UIPageViewController.TransitionStyle = .scroll,
    showsIndicators: Bool = true,
    @ViewBuilder // 👈 This is important
    content: () -> ForEach<Data, ID, Page>
  ) where Data.Element: Identifiable {
    self.init(
      content: .init(content()),
      axis: axis,
      transitionStyle: transitionStyle,
      showsIndicators: showsIndicators
    )
  }
}

Or if you don't want to use that extension you can be very explicit with your types:

  var body: some View {
    PaginationView(
      axis: .vertical,
      showsIndicators: false
    ) { () -> ForEach<Array<StoreOf<Page>>, ObjectIdentifier, PageView> in
      ForEach(Array(store.scope(state: \.pages, action: \.pages))) { store in
        PageView(store: store)
      }
    }
  }

[marcuswu0814]

Hi @mbrandonw, it work like a charm, thanks for your quick help, it save me lots of time. 😊

I've also created a PR at SwiftUIX, hoping it can assist anyone in the future.

[vmanot]

@mbrandonw @marcuswu0814 thank you so much for the PR! Just merged it in. I'll be cutting a new release later this week, feel free to use the tip of master for now.