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

Clarify when stream stats were updated on show page #788

Open
ggeisler opened this issue May 6, 2022 · 3 comments
Open

Clarify when stream stats were updated on show page #788

ggeisler opened this issue May 6, 2022 · 3 comments

Comments

@ggeisler
Copy link
Contributor

ggeisler commented May 6, 2022

On the stream show page, we display both when the stream was last updated and some files/records stats. The stream update info is always accurate but the stats are only generated once a day so are not necessarily coordinated with the stream update date/time.

Screen Shot 2022-05-05 at 3 43 38 PM

We need to make sure the user understands that if a stream has been updated since the files/records stats were generated, they might not be accurate. But we also need to coordinate how this is done with other changes being made to the stream metadata box.

@ggeisler
Copy link
Contributor Author

Below are a couple of potential options for ensuring the user understands that while the stream last updated info is always current, the stream statistics in the stream metadata box are only updated once a day.

(Note that in both cases the mockup changed the "Last updated:" text in the stream metadata box to "Stream updated:". That's not directed related to this issue, but it's a change I think we should make, either as part of this issue or just as a separate PR.)

Option A: Add a tooltip to provide explanatory text

Since we've already introduced the Tooltip code into the application, one solution is to simply use a tooltip to provide the update info:

Default state with a new tooltip added to the "Stream update" box header

Add a tooltip trigger as an info icon to the end of the stream metadata box heading:

tooltip-default


User-triggered tooltip state

And when triggered by the user, display the explanatory text in the tooltip:

tooltip-showing


The benefit of the tooltip option is it is relatively unobtrusive and doesn't negatively affect the page layout. A disadvantage might be that it requires the user to trigger the tooltip to see the explanatory text, but that's a pretty easy action to take, and likely not something a given user would need to do very often.

For either option I think we can use the same text we're using on the Providers page to reinforce this is the same issue with stats only being generated once a day: Statistics are calculated daily and might not reflect the most recent uploads.

Option B: Add the explanatory text directly to stream metadata box

This option makes the explanatory text harder to miss, but increases the vertical size of the box (and thus the header area in general, pushing the main page content a bit lower), and adds visual clutter. I'm a bit concerned that this text is distracting and would make it harder for the user to quickly scan the stream stats. I'm not sure the benefit of being hard to miss is worth that cost, given that once a user understands the situation they likely don't need to see that text again.

inline-text

@ggeisler ggeisler removed their assignment May 20, 2022
@thatbudakguy
Copy link
Member

I like option A since it keeps the box to a reasonable size.

@corylown
Copy link
Member

I'm in favor of option A as well.

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

No branches or pull requests

3 participants