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

reuse previous scrolling state when opening Devices from Details tab #871

Closed
vdelendik opened this issue Jan 16, 2024 · 4 comments
Closed
Milestone

Comments

@vdelendik
Copy link
Contributor

with new sorting functionality we display devices based on user input without automatic jumps...
at the same time after clicking to "Devices" tab we always scroll to the top automatically.

it would be great to remember previous scrollbar state to jump onto the same state and see reently occupied device. Problem might be discovered only on the farms with 50-70 devices when scroll is displayed on the Details tab

steps:

  1. login into any farm where we have 50+ devices
  2. activate Details tab
  3. scroll down for several pages and occupy device
  4. Click onto the global Devices button
    -> we jump onto the top and to see currently occupied device we have to scroll down

Expected behavior
-> Previous scroll bar state should be used and recently activated device should be visible

@vdelendik vdelendik added this to the 2.7 milestone Jan 16, 2024
ignacionar pushed a commit to ignacionar/stf that referenced this issue Feb 20, 2024
@ignacionar
Copy link
Collaborator

Done: #893

Details tab scroll bar state should remain now.

dhreben added a commit that referenced this issue Feb 22, 2024
#871 store user scrolling on details
dhreben added a commit that referenced this issue Feb 26, 2024
#871 Reuse scrolling status on reload and $location changes
@dhreben
Copy link
Contributor

dhreben commented Feb 26, 2024

@amastej available for testing
implemented new:
mcloud-device:2.6.4-beta11
STF:2.6.4-beta11

@dhreben dhreben closed this as completed Feb 26, 2024
@dhreben
Copy link
Contributor

dhreben commented Feb 29, 2024

@amastej available for testing
implemented new:
mcloud-device:2.6.4-beta17
STF:2.6.4-beta17

@amastej
Copy link

amastej commented Mar 4, 2024

Works with a slight delay on some devices

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Archived in project
Development

No branches or pull requests

4 participants