Floating file header jumps to top on sorting #15438

Activity

1. tobiasKaminsky

   added

   bug

   0. Needs triagePending check for reproducibility or if it fits our roadmapPending check for reproducibility or if it fits our roadmap

   on May 8, 2019
2. MorrisJobke

   added this to the Nextcloud 17 milestone on May 9, 2019
3. rullzer

   modified the milestones: Nextcloud 17, Nextcloud 17.0.1 on Sep 26, 2019
4. rullzer

   modified the milestones: Nextcloud 17.0.1, Nextcloud 17.0.2 on Nov 7, 2019
5. rullzer

   modified the milestones: Nextcloud 17.0.2, Nextcloud 17.0.3 on Dec 19, 2019
6. rullzer

   modified the milestones: Nextcloud 17.0.3, Nextcloud 17.0.4 on Jan 29, 2020
7. rullzer

   modified the milestones: Nextcloud 17.0.4, Nextcloud 17.0.5This milestone has been deleted on Mar 11, 2020
8. rullzer

   modified the milestones: Nextcloud 17.0.5This milestone has been deleted, Nextcloud 17.0.6 on Mar 23, 2020
9. 17 remaining items

   Load more
10. szaimen

    added

    26-feedback

    and removed

    needs info

    on Jan 11, 2023
11. skjnldsv

    added

    1. to developAccepted and waiting to be taken care ofAccepted and waiting to be taken care of

    and removed

    0. Needs triagePending check for reproducibility or if it fits our roadmapPending check for reproducibility or if it fits our roadmap

    on Mar 1, 2023
12. skjnldsv

    self-assigned this

    on Mar 1, 2023
13. skjnldsv commented on Mar 2, 2023

    

    skjnldsv

    on Mar 2, 2023

    Member

    More actions

    What should be the expected behaviour btw, if you sort everything, keeping the scrolling position would most likely not show the relevancy the user is looking for?
14. tobiasKaminsky commented on Mar 2, 2023

    

    tobiasKaminsky

    on Mar 2, 2023

    MemberAuthor

    More actions

    instead it position should remain, but only content should change.

    This is what I am used to on other file listings, e.g. Dolphin on Linux, or Thunderbird.
15. skjnldsv commented on Mar 2, 2023

    

    skjnldsv

    on Mar 2, 2023

    Member

    More actions

    Good from me 🤷‍♂️
    @jancborchardt @nimishavijay, you good with that?
16. nimishavijay commented on Mar 2, 2023

    

    nimishavijay

    on Mar 2, 2023

    Member

    More actions

    🤔 what @skjnldsv said about scrolling to show relevant info makes sense to me as well, on checking other apps it truly is a mixed bag. Windows file explorer and Google Drive scroll on changing sort order, while Dropbox, Nautilus on Ubuntu and the other apps @tobiasKaminsky mentioned don't scroll. I would go with scrolling for same reason @skjnldsv mentioned, ccing @jancborchardt here
17. jancborchardt commented on Mar 2, 2023

    

    jancborchardt

    on Mar 2, 2023

    Member

    More actions

    We do a hard refresh on re-sorting, so pretty difficult to keep the position anyway?

    I would also agree with @skjnldsv and @nimishavijay’s research in that it makes a bit more sense to scroll to top since the whole content changed.

    A quick animation that scrolls to the top of the list would of course be nice, but yeah with the hard refresh it’s difficult.
18. skjnldsv commented on Mar 2, 2023

    

    skjnldsv

    on Mar 2, 2023

    Last edited by skjnldsv

    Member

    More actions

    We do a hard refresh on re-sorting, so pretty difficult to keep the position anyway?

    There is some options possible. It's easy to remember the top-to-scroll-position pixel amount, but depending on what we wanna achieve, some stuff can and can't be done :)

    What about this:

    1. If a file is focused, we scroll to its position on a sorting change
    2. If a file is selected (or a group) we scroll to the first selected item
    3. If there is is none, we scroll up top
19. skjnldsv commented on Mar 2, 2023

    

    skjnldsv

    on Mar 2, 2023

    Member

    More actions

    A quick animation that scrolls to the top of the list would of course be nice, but yeah with the hard refresh it’s difficult.

    Also, keep in mind files to vue is still ongoing, a hard refresh might be avoidable here (just re-rendering might be possible)
20. skjnldsv

    closed this as completedin #39808on Aug 17, 2023
21. skjnldsv

    added this to Files to vueon Aug 7, 2024
22. skjnldsv

    moved this to Done in Files to vueon Aug 7, 2024