NetNewsWire icon indicating copy to clipboard operation
NetNewsWire copied to clipboard

Categories are not being displayed

Open s22-tech opened this issue 8 months ago • 4 comments

Is there a way to have NNW display the categories of a feed?

In case the feed is set up incorrectly, here's a portion of one:

<?xml version="1.0" encoding="utf-8" ?>
<rss version="2.0">
  <channel>
    <title>LiteCart Add-Ons</title>
    <link>https://www.litecart.net/?</link>
    <description></description>
    <language>en</language>
    <item>
      <title><![CDATA[PayPal Payment]]></title>
      <link>https://www.litecart.net/en/addons/779/paypal-payment</link>
      <description><![CDATA[PayPal Payment Module for LiteCart]]></description>
      <category><![CDATA[Payment Modules]]></category>
      <pubDate>Sat, 10 May 2025 00:27:09 +0000</pubDate>
    </item>
    <item>
      <title><![CDATA[Change Order Status]]></title>
      <link>https://www.litecart.net/en/addons/76/change-order-status</link>
      <description><![CDATA[Change order status for multiple orders in admin]]></description>
      <category><![CDATA[Admin Apps]]></category>
      <pubDate>Sun, 04 May 2025 19:00:16 +0000</pubDate>
    </item>
    <item>
      <title><![CDATA[Trumbowyg Upload Image]]></title>
      <link>https://www.litecart.net/en/addons/211/trumbowyg-upload-image</link>
      <description><![CDATA[Upload images using the Trumbowyg editor.]]></description>
      <category><![CDATA[Core Features]]></category>
      <pubDate>Sat, 26 Apr 2025 20:14:33 +0000</pubDate>
    </item>
    ...
  </channel>
</rss>

In the screenshot, you can see that the category "Admin Apps" is not shown.

Image

Setup: NNW v6.1.9 macOS 15.3.1

P.S. Nice reader! I'm really impressed with it. Thanks!

s22-tech avatar May 10 '25 14:05 s22-tech

I couldn't spot any template variable for categories: https://github.com/Ranchero-Software/NetNewsWire/blob/49efb15be5c36759420c0be62449ab85deb62113/Shared/Resources/Promenade.nnwtheme/template.html#L2-L32

Zodiac1978 avatar May 19 '25 11:05 Zodiac1978

NetNewsWire doesn’t parse or store categories.

My thinking is that they’re not actually useful except in rare cases, and so it’s not worth the time to parse categories and especially not worth the time + storage to save categories in the database.

I could be wrong! If you know of good reasons to support categories, please tell me!

brentsimmons avatar May 22 '25 02:05 brentsimmons

Well, in the case I gave above, displaying the categories would give us more information as to whether or not we want to click through to read the post. A lot of times, the name or the description isn't descriptive enough since this site allows the module's authors to add that info, and many of them don't speak English well. The categories are assigned by the site, so those are set in stone.

It looks like you're including everything but the categories, so maybe adding those wouldn't be too much of a burden on the db?

s22-tech avatar May 22 '25 03:05 s22-tech

Parsing wouldn’t be much burden — but it would definitely add to save and fetch time for the database and it would make the database larger.

To quantify this would actually require writing the feature and then testing with and without the feature. My best guess is that fetches would be 10% slower, and the database about 20% larger. (Totally just a guess, though.)

brentsimmons avatar May 22 '25 04:05 brentsimmons