r4dixx / theguardianview Goto Github PK

Template here

List items could show a bit more detail. Including the article thumbnail and publication date would make the app more engaging. This wouldn't take too long as the logic is already there for title, description and URL.

Ideally, thumbnails would be positioned on the left side of list items and date on the right side, top corner.

Changes should be reflected immediately. Now it is necessary to kill the app and open it again.

Preliminary actions = "ArticleActivity → SettingsActivity → performing changes (e.g. API key)"

Current behavior:

Preliminary actions → ArticleActivity with same state than before

Preliminary actions → reducing the app and reopening the app → ArticleActivity with same state than before (changes are still not reflected)

Preliminary actions → killing the app and opening it again → ArticleActivity with new state (changes are reflected)

Expected behavior:

Preliminary actions → ArticleActivity with new state (no matter if the app gets reduced or killed)

Bug introduced by f876c16

As already suggested in r4dixx/VisitCalgary#3 and r4dixx/RookiePlayer#7 and probably every repo of mine.

This feature isn't working properly. Button sure get randomized, but they should remain attach to their list item. In other words, refreshing the app or closing/opening it shouldn't make the button text change. Button texts should change at the same rate to which list items change.

Current Behavior:
It blindly refreshes no matter what

Expected Behavior
ListView should only refresh when data has changed. It is better UX-wise and performance-wise

Let's say I'm in the middle of the ListView (around the 25th item). When I click the list item, browser opens and I can read the article. So far so good. Now let's say I've finished reading and want to read the next/previous one, I'm hitting the back button aaaand... I'm at the top of the list. This is annoying, I should get back where I left off. Similarly, that same behavior should happen when leaving the app, locking phone, etc.

It is kind of working so far, but it's not perfect. Here are a few cases where it needs to be fixed:

ListView blinks/flickers on refresh
Current Behavior:
Refreshing makes the ListView blink
Expected Behavior:
Refresh action should be smooth. Either add animations or remove the blinking altogether.
FIXED with 5751615

ListView refreshes even when data didn't change

• Current Behavior:
  It blindly refreshes no matter what, which doesn't help with the blinking (FIXED with 5751615)
• Expected Behavior
  ListView should only refresh when data has changed. It is better UX-wise and performance-wise

Empty states are not handled correctly

• Current Behavior:
  App is open and data loaded. Network connection becomes inexistant (offline). Refreshing doesn't lead to the "no internet" view. EDIT for clarification, it leads to the "no articles" view instead
  Similarly, app is open, data is not loaded because network connection is off. When it comes back, refreshing doesn't do anything.
  The same applies to the "no articles" view
  FIXED with 5751615
• Expected Behavior:
  Refreshing should refresh. Refreshing should update to the current state of the app i.e. show the appropriate view (listview or no internet connection)

Current behavior: after preference changes, the listview remains at the same position. This is annoying particularly when changing sort order.

Expected behavior: if any change has been made on both settings, the ListView should go back to position 0.

Could be better UX wise to know which ones are the latest at a glance. Could be the first list item or the articles published today or, even better, the X articles published since app was last open. To think through.

See r4dixx/VisitCalgary#6