search

mboinet / ttrss-mobile

A mobile webapp for Tiny Tiny RSS
GNU Affero General Public License v3.0
189 stars 28 forks source link

resolving issue 29 : Hide feeds & categories with no new articles #51

Closed alexandrenorman closed 11 years ago

alexandrenorman commented 11 years ago

It works but saving preferencies seems to be bugged... Anyway, it needs to reload the page to be effective.

mboinet commented 11 years ago

Hi!

I didn't test this yet, but I would have done this differently.

Instead of hiding with CSS categories, I would have prevented the model to request empty categories. getCategories API call has an include_empty boolean to do this. When leaving the settings page, the list should be updated this way.

Tell me if you want to work on this.

alexandrenorman commented 11 years ago

Hello,

I didn't checked TTRSS api. Indeed your solution is much cleaner. Here is the new code including API use and removing CSS quick hack.

mboinet commented 11 years ago

I'll test this and try to integrate this after the 1.0 release.

mboinet commented 11 years ago

Almost ready I think, see my hideEmpty branch and https://github.com/alexandrenorman/ttrss-mobile/pull/2

mboinet commented 11 years ago

I fixed a few things, it's working great. It's merged in master.