Fix back navigation on file page #416

Closed
opened 2019-01-29 16:59:06 +01:00 by tzarebczan · 2 comments
tzarebczan commented 2019-01-29 16:59:06 +01:00 (Migrated from github.com)

The Issue

When you go to a file page, clicking back always take you back to explore. This is not expected when you are coming to the file page from a channel or subscriptions.
From Akin:
For going back to discover, that's because the file page is designed to always return to discover because they're in the same "stack" navigator. I'll see if there's a way I can alter this behaviour when the start point is Trending. For the Prev / Next buttons, the back button is not connected at all to this.

System Configuration

  • Device:
  • Android version:

Anything Else

Screenshots

Internal Use

Acceptance Criteria

Definition of Done

  • Tested against acceptance criteria
  • Tested against the assumptions of the user story
  • The project builds without errors
  • Unit tests are written and passing
  • Tests on devices/browsers listed in the issue have passed
  • QA performed & issues resolved
  • Refactoring completed
  • Any configuration or build changes documented
  • Documentation updated
  • Peer Code Review performed
<!-- Thanks for reporting an issue to LBRY and helping us improve! To make it possible for us to help you, please fill out below information carefully. Before reporting any issues, please make sure that you're using the latest version. We are also available on live chat at https://chat.lbry.io --> ## The Issue When you go to a file page, clicking back always take you back to explore. This is not expected when you are coming to the file page from a channel or subscriptions. From Akin: ```For going back to discover, that's because the file page is designed to always return to discover because they're in the same "stack" navigator. I'll see if there's a way I can alter this behaviour when the start point is Trending. For the Prev / Next buttons, the back button is not connected at all to this.``` ## System Configuration - Device: - Android version: ## Anything Else <!-- Include anything else that does not fit into the above sections --> ## Screenshots <!-- If a screenshot would help explain the bug, please include one or two here --> ## Internal Use ### Acceptance Criteria 1. 2. 3. ### Definition of Done - [ ] Tested against acceptance criteria - [ ] Tested against the assumptions of the user story - [ ] The project builds without errors - [ ] Unit tests are written and passing - [ ] Tests on devices/browsers listed in the issue have passed - [ ] QA performed & issues resolved - [ ] Refactoring completed - [ ] Any configuration or build changes documented - [ ] Documentation updated - [ ] Peer Code Review performed
robvsmith commented 2019-03-04 22:35:34 +01:00 (Migrated from github.com)

This came up in the reviews on BetaTesting.com a few times. Making a note here instead of making a new ticket.

This came up in the reviews on BetaTesting.com a few times. Making a note here instead of making a new ticket.
akinwale commented 2019-03-06 16:57:54 +01:00 (Migrated from github.com)

Fixed.

Fixed.
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: LBRYCommunity/lbry-android#416
No description provided.