Only Works with List

[Peter-Schorn]

This only seems to work with lists. There are many cases where I cannot use a list but would still like to enable pull down to refresh. For example, how would I enable this on a ScrollView?

[Bubba8291]

I'm having the same issue too. +1

[Peter-Schorn]

@Bubba8291 I was actually able to figure out how to get it to work with scroll views. You can find my fork here: https://github.com/Peter-Schorn/SwiftUIRefresh

[Bubba8291]

@Peter-Schorn Unfortunately, it didn't work for me. Are you on Xcode beta? If so, which one. I'm on Xcode B3, so maybe that's why.

[Peter-Schorn]

I just tested in in beta 2 (12A6159), and I have found that when you attach pullToRefresh as a modifier to the ScrollView, it doesn't work, but when you attach it to a VStack that is the direct child of a ScrollView, then it does work, although the position of the activity indicator is higher up than it should be.

This works:

ScrollView {
    VStack {
        ForEach(data, id: \.self) { item in
            Text(item)
                .padding()
        }
    }
    .pullToRefresh(
        isRefreshing: $isRefreshing,
        onRefresh: onRefresh
    )
}

But this does not:

ScrollView {
    VStack {
        ForEach(data, id: \.self) { item in
            Text(item)
                .padding()
        }
    }
}
.pullToRefresh(
    isRefreshing: $isRefreshing,
    onRefresh: onRefresh
)

[Bubba8291]

The code worked in Xcode B3 for me.

[Peter-Schorn]

@Bubba8291 Did you have the same issue with the activity indicator Benning to high up on the screen?

[RobertPeteleaza]

I just tested in in beta 2 (12A6159), and I have found that when you attach pullToRefresh as a modifier to the ScrollView, it doesn't work, but when you attach it to a VStack that is the direct child of a ScrollView, then it does work, although the position of the activity indicator is higher up than it should be.

This works:

ScrollView {
    VStack {
        ForEach(data, id: \.self) { item in
            Text(item)
                .padding()
        }
    }
    .pullToRefresh(
        isRefreshing: $isRefreshing,
        onRefresh: onRefresh
    )
}

But this does not:

ScrollView {
    VStack {
        ForEach(data, id: \.self) { item in
            Text(item)
                .padding()
        }
    }
}
.pullToRefresh(
    isRefreshing: $isRefreshing,
    onRefresh: onRefresh
)

I tried your suggested method. It doesn't work for me, unfortunately.

[bradley]

I ended up writing an alternative version, but somewhat based on this repo, for my ScrollView needs. I've heavily tested it in numerous use cases such as heavy styling of the content view and such as being wrapped in problematic UI like a NavigationView. This could likely be easily modified to work for list views too should other bugs herein block you.

https://gist.github.com/bradley/7517e6592a3621bf57aa5dbb7395e394