search

cnlpete / ttrss

ttrss is a Tiny Tiny RSS Reader App for the Nokia N9 smart phone, written using Qt/QML. ttrss is using the Tiny Tiny RSS API.
http://ttrss.cnlpete.de
GNU General Public License v2.0
30 stars 12 forks source link

[sailfish] “Load More” Button missing #124

Closed michael-k closed 10 years ago

michael-k commented 10 years ago

How to reproduce:

  1. Set ordering to “Oldest first”.
  2. Go to a feed with > 200 articles.
  3. Only the 200 newest articles are displayed and there is no “Load More” Button.

The problem with this is, that you look at all visible items, think you've read the whole feed (no one's counting 200 items) and do a “mark all read”. This way information gets lost.

I don't know about Harmattan.

michael-k commented 10 years ago

The button is on purpose only shown for ordering “Newest first”.

Possible solutions:

  1. Show a similar button in the header.
  2. Load the oldest items first and make the button in the footer useable (for “Oldest first”). This might not only work if read items are shown, because we don't know were to start loading.
cnlpete commented 10 years ago

hmmm ... in harmattan i have the load more button in the header (similar to your solution 1)

solution 1 should be more simple, but in sailfish it is kind of hackish as the header has the PageHeader already. Quick testing did not reveal a good option for this. do you have any ideas?

solution 2 would bring a few problems, for example when showing all feeditems, do we really want to scroll through all existing items? those would be a lot when using the software for some time....

we could also simply show the existing "load more" button at the end of the list, as the user wants to know that there are more entries when he is finished reading (= at the end of the list) i am however not sure where the items would be appended or how to properly handle this ...

michael-k commented 10 years ago

I prefer solution 1. The header can be used for more than just the PageHeader. Example: https://github.com/cnlpete/ttrss/blob/master/qml/ttrss/sailfish/pages/LabelUpdater.qml#L43

Btw.: Why are we not using the order_by parameter of getHeadlines?

cnlpete commented 10 years ago

ok, looks simple and usable :)

Btw.: Why are we not using the order_by parameter of getHeadlines?

Because it wasn't available back when i started development :) we could/should use that option instead of manually sorting