Ranchero-Software / NetNewsWire

RSS reader for macOS and iOS.
https://netnewswire.com/
MIT License
8.28k stars 526 forks source link

Unable to add feed alert wording #3799

Open stuartbreckenridge opened 1 year ago

stuartbreckenridge commented 1 year ago

We have two very different alert titles and messages when NetNewsWire cannot add feeds when they are not found.

In this example, the website was bbc.com/news:

Mac:

iOS:

I'd propose bringing consistency to them:

Screenshot 2022-12-30 at 07 11 46

IMG_227DA08E4DA3-1

brentsimmons commented 1 year ago

I like the title “Unable to Add Feed.”

Error text should almost always be of the form “Can’t do x because y.” So the text should remain:

“Can’t add a feed because no feed was found.”

stuartbreckenridge commented 1 year ago

Can I fix this in the internationalization PR?

brentsimmons commented 1 year ago

Yes — good thinking.

brentsimmons commented 1 year ago

(Closed by accident. Oops!)

stuartbreckenridge commented 1 year ago

This is fixed in ceb925d.

Note that this commit localises AccountError in the Account package, and both iOS and macOS now use those strings. This means that the iOS / macOS apps don't need to have independent copies of those strings in their respective strings files.

stuartbreckenridge commented 1 year ago
Screenshot 2023-03-11 at 17 26 05 Screenshot 2023-03-11 at 17 26 45