ccomeaux / boardgamegeek4android

BoardGameGeek application for Android (unofficial)
GNU General Public License v3.0
228 stars 47 forks source link

Loading state displays on top of search screen content #90

Closed kamal-kamalmohamed closed 6 years ago

kamal-kamalmohamed commented 6 years ago

While searching for a game the loading state displays on top of the list content.

Scenario 1: Steps:

Expected:

Actual:

Scenario 2: Steps:

Expected:

Actual:

Reported on Android 8.0. Attached screenshots.

In terms of how to fix this I would suggest an horizontal loading state at the top of search results instead of having a full screen one. I'll put together a PR if that's alright.

screenshot_20180603-120437 screenshot_20180603-153017

kamal-kamalmohamed commented 6 years ago

@ccomeaux Happy with the suggested approach of having an horizontal progress bar just below the search bar? Happy to submit a PR for it. 😄

ccomeaux commented 6 years ago

The progress bar is working as intended, but I'm not sure my intention is better than your suggestion. So fire away!

kamal-kamalmohamed commented 6 years ago

Raised PR here: https://github.com/ccomeaux/boardgamegeek4android/pull/92

ccomeaux commented 6 years ago

Will be in next minor release