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

"Last refresh:" activity status message is not treated as default status #414

Closed
sssoleileraaa opened this issue Jun 10, 2019 · 3 comments
Closed
Labels
bug Something isn't working

Comments

@sssoleileraaa
Copy link
Contributor

Description

When downloading a file, you'll see the "Downloading file..." message in the activity status bar. After it finishes, the "Last refresh" message does not reappear. It's blank until the next 5-minute refresh happens.

STR

  1. Download a file and see message in status bar
  2. Wait until message goes away (~10 seconds)
  3. See no "Last refresh" until the 5-minute sync occurs or until you just manually click the refresh button

Expected behavior

By default, we should always show the last time a refresh occurred.

Idea

  • The ActivityStatusBar could keep a timer for each message and emit a signal when the message times out to the controller. Then the controller could grab the last time since a refresh and set the status of the activity bar.
@eloquence eloquence added the bug Something isn't working label Jun 11, 2019
@eloquence
Copy link
Member

Confirmed in local dev env.

@eloquence eloquence changed the title No default "Last refresh..." message "Last refresh: ..." activity status message is not treated as default status Jun 11, 2019
@eloquence eloquence changed the title "Last refresh: ..." activity status message is not treated as default status "Last refresh:" activity status message is not treated as default status Jun 11, 2019
@eloquence eloquence added this to the 0.2.0alpha milestone Jul 2, 2019
@eloquence eloquence modified the milestones: 0.2.0alpha, 0.2.0beta Dec 12, 2019
@eloquence
Copy link
Member

eloquence commented Dec 19, 2019

In the UX meeting today, we discussed significant potential revisions to the refresh widget, largely to ensure the UI remains consistent with the changes described in #652. In addition to changing the refresh widget to an activity indicator, the specific proposal we discussed was to show a timestamp only in case the client is offline or experiencing network issues, since the client can otherwise be expected to constantly poll the server for new messages/replies.

This issue is not currently actionable, so removing from the backlog for now (but kept open because it's still a bug); added a card to create a spec for the intended behavior.

@eloquence
Copy link
Member

No longer applicable due to changed behavior, as foreshadowed above.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants