BiglySoftware / BiglyBT-plugin-scanerss

ScaneRSS is a plugin application that reads RSS feeds and allows for automatic torrent download from these feeds by use of filters (rule sets).
3 stars 1 forks source link

Feed icons being rendered WAY to large. #3

Open koshell opened 2 years ago

koshell commented 2 years ago

I'm having an issue where the RSS feed icons are being rendered comically large: image I think this is due to one of the RSS feeds returning a massively oversized icon and all the other feeds are being upscaled to match. Is it possible to add some logic to force downscale the icons to something sane regardless of what the size of the returned icon is? As it stands this renders the Main View unusable since the offset and row size all seem to be affected.

parg commented 2 years ago

Did you say the issue resolved itself? This plugin isn't really supported anymore since the author moved on and I would have to try and figure it out and I'm lazy...

koshell commented 2 years ago

Kinda, I changed the RSS to one on the same site that didn't have a icon but didn't realise at the time. I tried re-adding the problematic RSS feed again and it isn't breaking this time. Maybe as long as the first feed has a correctly sized icon it doesn't break? I'm unsure how to trigger it again and I'm not super keen on messing up all my RSS feeds to try to replicate it.

At this point if I had to guess I would say somewhere in the code it is sampling the dimensions of the icon to scale the row height. I would probably leave this issue open in case someone else trips over it. But at least in my case adding a few more RSS feeds before re-adding the problematic one seems to be a workaround.