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

Activities design review #5933

Closed
4 tasks done
camilasan opened this issue Jul 27, 2023 · 14 comments · Fixed by #6064
Closed
4 tasks done

Activities design review #5933

camilasan opened this issue Jul 27, 2023 · 14 comments · Fixed by #6064
Assignees
Labels

Comments

@camilasan
Copy link
Member

camilasan commented Jul 27, 2023

  • File sharing: not all files have the option to share or view details (bug?)

  • The progress bar for 'sync now' is too high and it should only show up if the sync will take more than ~2 seconds. Only the spinner is enough when it is so quick.

  • Show 'Synced' activities: only things you did locally, not changes from the server (what comes back as activities)

  • when new item, scroll automatically to top

@jancborchardt
Copy link
Member

To add:

  • Also investigate if it is possible to show the notification for "Log in via notification" quicker. Currently the possible wait time is up to 5 minutes cc @mgallien

@tobiasKaminsky
Copy link
Member

Show icons: nextcloud/server#16207

@allexzander
Copy link
Contributor

@camilasan @jancborchardt @tobiasKaminsky

For:
- File sharing: not all files have the option to share or view details (bug?)

We do NOT display the dots button for activities that are not files or folders, also for those that are for action "file_deleted", and for those that do not have a path that we can use to open them locally.

Do you have any examples of false positives for mentioned scenarios?

@allexzander
Copy link
Contributor

@jancborchardt Should this happen always?

  • when new item, scroll automatically to top

I can already expect how the user might get annoyed when in search for one of the older items and not being to reach it cause the auto-scrolling is happening during intensive sync/lots of notifications from the server

@allexzander
Copy link
Contributor

allexzander commented Sep 8, 2023

@jancborchardt as for:

  • The progress bar for 'sync now' is too high and it should only show up if the sync will take more than ~2 seconds. Only the spinner is enough when it is so quick.

~2 seconds sounds a bit vague and unreliable + too much hardcoded, there can be shorter or longer intervals before sync actually starts, which will result in inconsistent behavior. What if we instead display the progress bar once we know that there are some files (at least 1) that will get synced, during the discovery? This will then look like that once the number is more than 0:
image

The downside is, for a few seconds (from ~5 to ~20), we will be seeing this:
image

Maybe, the text should be changed to something like "Preparing sync" or "Discovering", while the scrollbar can not be displayed. What do you think?

Also, what do you think of progress bar height now? Does it look not too high anymore on my screenshot?

@jancborchardt
Copy link
Member

jancborchardt commented Sep 11, 2023

@allexzander here’s my feedback :)

We do NOT display the dots button for activities that are not files or folders, also for those that are for action "file_deleted", and for those that do not have a path that we can use to open them locally.
Do you have any examples of false positives for mentioned scenarios?

Yeah, actually when opening the system tray, most of the sync entries do not seem to have a 3 dot menu, like here (there’s many more entries below like this):
image

Should this happen always?

when new item, scroll automatically to top
I can already expect how the user might get annoyed when in search for one of the older items and not being to reach it cause the auto-scrolling is happening during intensive sync/lots of notifications from the server

No, this is specifically about: When the view was previously scrolled to the top (or within a few pixels of it, probably good to add that as a buffer), then new notifications should continue the autoscrolling.
The view should indeed not scroll if you are down in the list and new stuff comes up on top. For that we could have a "New activity" bubble on the top like we have for "Unread mentions" in Talk:
image

~2 seconds sounds a bit vague and unreliable + too much hardcoded, there can be shorter or longer intervals before sync actually starts, which will result in inconsistent behavior. What if we instead display the progress bar once we know that there are some files (at least 1) that will get synced, during the discovery?

Right, this sounds good, as well as changing the initial text to "Preparing sync".
Could you make the progress bar height and border-radius look like the HTML progress bar? https://developer.mozilla.org/en-US/docs/Web/HTML/Element/progress

@allexzander
Copy link
Contributor

@jancborchardt
Here is how the new progress bar with rounded borders looks:
image
Hope, this is ok.

@jancborchardt
Copy link
Member

@allexzander much better! Can the filled area also be a different color, like primary of the operating system, or of the Nextcloud?

@allexzander
Copy link
Contributor

allexzander commented Sep 12, 2023

@jancborchardt Hope the below color is fine, it will be the same as the header when it is changed (like, if the header becomes yellow, this will also become yellow, and so on):
image

@allexzander
Copy link
Contributor

allexzander commented Sep 13, 2023

@jancborchardt

No, this is specifically about: When the view was previously scrolled to the top (or within a few pixels of it, probably good to > add that as a buffer), then new notifications should continue the autoscrolling.
The view should indeed not scroll if you are down in the list and new stuff comes up on top. For that we could have a "New > > activity" bubble on the top like we have for "Unread mentions" in Talk:

What do you think? This button disappears over 2.5 seconds if not hovered such that it does not hang there indefinitely. When hovered - disappearing gets canceled but restarts again when the mouse leaves the button. It scrolls to the top when clicked.

button_for_new_notifications.mp4

@allexzander
Copy link
Contributor

I am wondering if we now even need the "when a new item, scrolls automatically to top" without clicking a button, or is the button enough for the user to be able to scroll only if needed?

@jancborchardt
Copy link
Member

@allexzander nice!

  • Progress bar looks good now!
  • The "New activities" button should go on the top, not bottom of the list since the new activities are up top :)
  • The disappearing looks a bit strange and takes long? I would say just keep it there for 5 seconds but fade it out faster
  • Yes we definitely need the autoscroll to top on new activities if you are already on top! :) By default people don't necessarily scroll down, and you shouldn't need to click a button all the time. ?

@allexzander
Copy link
Contributor

@jancborchardt
Here you go, the button is fading for 1.5 seconds to 0.25 opacity and then disappearing after 5 seconds (if not hovered), see the video:
https://github.com/nextcloud/desktop/assets/12224014/420e11d3-c1a0-4074-ac04-d9c70722c544

@nimishavijay
Copy link
Member

nimishavijay commented Sep 14, 2023

Looks nice already! :) I have 2 points from my side:

  • With the 0.25 opacity I am worried about accessibility, because the contrast doesn't seem like it's enough. I would say that it can stay full opacity, and it not hovered for 5 seconds it could disappear
  • Since it is scrolling to top, we could add this up arrow icon (expand-less on MDI) to the left of the button (nice-to-have, but not a blocker for this PR)

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

Successfully merging a pull request may close this issue.

6 participants