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`.
Noticed that the window size wasn't returning the new value and that's why using the window size to calculate the size to restore it to we were incrementing the original size by one.
Not sure if this breaks the fix that this bigSurOffsetFix was implementing though because I can't replicate that on macOS 12.4
If NetNewsWire says it is able to handle http and https URL schemes, Safari will list NetNewsWire as a target in Develop > Open Page With, which provides a trivial flow for subscribing to the page or site one is currently reading in Safari. I believe this functionality existed in NetNewsWire "classic," and this patch just brings the functionality back.
Remove cached `draggedNodes` optimization and just always use the data on the dragging pasteboard because it works both within the same window and across two windows.
Add support for WebFeedPasteboardWriter to include a parent containerName in the pasteboard dictionary written for a WebFeed object (which doesn't know about it's parent) because we need this for drag and drop to be able to perform a move operation.
Fix issue with dragged feeds not knowing their parent folder and delete deleting the wrong item by telling WebFeedPasteboardWriter what the container of items is. Remove code that is now unused now that draggedNodes is removed and consolidate code that was expanded to handle both node and pasteboard items since we are just using pasteboard items now.
As it may seem very strange to some people that I'm creating a pull request to fix such a non-issue, here's the explanation: I'm working on a tool that reads Strings files (similar to https://github.com/Flinesoft/BartyCrouch) and my Strings file parser fails in my "real-world project tests" due to this comment. 😅
