renun/NetNewsWire
1
0
Fork 0
mirror of https://github.com/Ranchero-Software/NetNewsWire synced 2025-08-12 06:26:36 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

Fix account add crash

Browse Source 
As reported in Ranchero-Software#3606, it was the case that attempting
to add an account could result in a crash. This fixes the problem by
retaining the `NSHostingController` that holds the `AddAccountsView`
view in a manner similar to how 48138b1bb retained
`accountsAddLocalWindowController`.
This commit is contained in:
Winnie Quinn
2022-09-03 17:43:16 -04:00
parent ed688fa789
commit 2e89423a2b
1 changed files with 2 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
Mac/Preferences/Accounts/AccountsPreferencesViewController.swift
View File

@@ -24,6 +24,7 @@ final class AccountsPreferencesViewController: NSViewController {
@IBOutlet weak var deleteButton: NSButton!
var addAccountDelegate: AccountsPreferencesAddAccountDelegate?
var addAccountWindowController: NSWindowController?
var addAccountsViewController: NSHostingController<AddAccountsView>?
private var sortedAccounts = [Account]()
@@ -52,6 +53,7 @@ final class AccountsPreferencesViewController: NSViewController {
let controller = NSHostingController(rootView: AddAccountsView(delegate: self))
controller.rootView.parent = controller
presentAsSheet(controller)
addAccountsViewController = controller
}
@IBAction func removeAccount(_ sender: Any) {