Dismissable popover in SwiftUI

Putting each item row in a subview will make the popover presentation much easier. Each view has its own state variable, you don't have to handle the id.

Here is a possible approach:

struct AppMenu: View { var itemList: [Item] = [Item(URL(string: "https://google.com")!)] var body: some View { VStack { List { ForEach(itemList, id: \.self) { item in ItemView(item: item) } } } } } struct ItemView: View { @ObservedObject var item: Item @Environment(\.openURL) var openUrl @State private var isLinkPresented = false var body: some View { Button(action: { isLinkPresented = true }) { Label(item.contentUrl.absoluteString.replacingOccurrences(of: "https://", with: ""), systemImage: "arrow.right.circle.fill") } .popover(isPresented: $isLinkPresented) { VStack { // Link("\(item.contentUrl.absoluteString)", destination: item.contentUrl) Button("Open in sharari") { // openURLInSafari(item.Content) // rename to "item.contentUrl" openUrl(item.contentUrl) // isLinkPresented = false // this is not neccessary } } .padding() } } } 

Setting the isPresented attribute of popover to false is not neccessary in SwiftUI. It will be automatically set when focus changes.

I made up your Item object, as it was not implemented in your example:

class Item: ObservableObject, Hashable { let id = UUID() let contentUrl: URL init(_ contentUrl: URL) { self.contentUrl = contentUrl } static func ==(lhs: Item, rhs: Item) -> Bool { return lhs.id == rhs.id } func hash(into hasher: inout Hasher) { hasher.combine(id) } } 

Note that you named a property Content, but properties should be llamaCased and meaningfully named, so I called it contentUrl, which makes it clearer what's going on.