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

Progress bar on collection page #251

Open
matsbov opened this issue Sep 25, 2024 · 8 comments
Open

Progress bar on collection page #251

matsbov opened this issue Sep 25, 2024 · 8 comments
Assignees
Milestone

Comments

@matsbov
Copy link
Contributor

matsbov commented Sep 25, 2024

On the Records tab of the collection page there is a progress bar showing how many of the total/estimated number of specimens in the collection are available in the LA portal. However, if the total number of specimens is not available it defaults to displaying the message "No records are available for viewing..." which is often wrong and produces a very confusing page where the left part says there are a lot of records available and the right part says no records:

Screenshot-2

This PR (#250) fixes this by simply hiding the progress bar if the total/estimated number of specimens is not available since the progress bar makes no sense at all in that case.

@adam-collins
Copy link
Contributor

This is a data entry issue.

This section of the Records tab shows the difference between records provided to the ALA vs the records in the collection.

To fix this, enter the number of records that the collection has into the correct field.

  1. Edit the collection.
  2. Find the "Edit" button beneath the title "Number of specimens in the collection".
  3. Enter the number of records in the collection into the "Number of specimens" field.
  4. Save the changes and check the Records tab of the collection.

For the given example I manually entered the number of records as the "Number of specimens" to confirm that this works.

@matsbov
Copy link
Contributor Author

matsbov commented Sep 26, 2024

Yeah, I am aware the progress bar works fine if there is data available. It's just that out of the 78 collections in our Collectory we have two where the number of specimens have been entered. We will never have that data for all collections. Just randomly clicking around the collections in ALA and some of the other LivingAtlases it's not uncommon to see this. So I still think it's better to hide the progress bar if the number of specimens have not been entered.

@adam-collins
Copy link
Contributor

That is useful context. I was approaching this as an ALA data entry issue and the PR would hide the problem rather than leaving an indication that something is missing.

I will reopen the PR while awaiting the data team's input into this issue.

@patkyn
Copy link

patkyn commented Oct 3, 2024

While the number of specimens need to be populated in order to see the progress bar for eg: https://collections.ala.org.au/public/show/co118, however, this information may not always be supplied by the data provider which we need to make awareness of this.
The message "No records are available for viewing in the Atlas of Living Australia" is rather misleading as it does not even indicate that the number of specimen is missing. I do agree that it does make sense to hide the progress bar in the UI perspective, however I think the wording need to be changed as well to show why the progress bar is not shown, something like "Specimen record count is not available to show the progress bar for records".

@adam-collins
Copy link
Contributor

adam-collins commented Oct 3, 2024

I'll merge the PR and add this alternate text.

I note that the old text that conveyed this was, at one time, "There is no estimate of the total number of {0} in this collection."

@peggynewman
Copy link

Is it a UI thing to give a reason why something is not showing? Don't you just not show it at all?

@adam-collins
Copy link
Contributor

Applying @patkyn's input the results in the following

@adam-collins
Copy link
Contributor

Test links are no longer working as the changes are approved and now in a PR.

@adam-collins adam-collins added this to the 5.2.0 milestone Oct 24, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants