Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

When you use the scroll bar in main app screen it also moves the view in download folder and vice versa #729

Closed
LavRadis opened this issue Nov 10, 2017 · 4 comments
Assignees
Labels
area: ux type: bug Existing functionality is wrong or broken

Comments

@LavRadis
Copy link
Contributor

LavRadis commented Nov 10, 2017

Working on this? See hint here

The Issue

When you scroll down in main app screen, the view in the "downloads and purchase"s is also moved when you go there. It also happens in reverse, when you move scroll in downloads view and go back to the main screen.

Steps to reproduce

  1. Move scroll down in the main screen
  2. Go to Downloads section
  3. You will notice that the view focus is not on top at last downloaded video, but it has moved down

It also works in reverse. Did not check for all app sections and views.

Expected behaviour

Moving scroll in one view should not affect other sections of the app.

Actual behaviour

You move scroll in one section, the focus moves in other also.

System Configuration

LBRY Daemon version: 0.18.0rc1
LBRY App version: 0.18.0rc1
LBRY Installation ID: 7tYN144rHXrspzJG9xLP28fi1JztXnbpKro6DBJEQyzBmXTyDgqRLfYsUvHYFVmMe2
Operating system: Win 10

Screenshots

They are attached to this issue report.

lbry view 1
lbry view 2

@LavRadis LavRadis changed the title When you use a scroll bar in main app screen it also moves the view in download folder and vice versa When you use the scroll bar in main app screen it also moves the view in download folder and vice versa Nov 10, 2017
@LavRadis
Copy link
Contributor Author

It also happens when you move scroll in the wallet. When you go to the main window it has also moved.

@LavRadis
Copy link
Contributor Author

It happens when you move the settings scroll bar also. Looks like all the areas of the app are affected.

@kauffj
Copy link
Member

kauffj commented Nov 10, 2017

Clue for whoever works on this:

You most likely to change the scrolling behavior in the doNavigate function of actions/navigation.js so that the Y position is not retained on navigation to new pages not in the history stack.

@kauffj kauffj added type: bug Existing functionality is wrong or broken contributor friendly labels Nov 10, 2017
@btzr-io
Copy link
Collaborator

btzr-io commented Nov 14, 2017

fixed in #732

@neb-b neb-b closed this as completed Nov 17, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: ux type: bug Existing functionality is wrong or broken
Projects
None yet
Development

No branches or pull requests

4 participants