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

Study details tab: update summary information header #143

Closed
3 tasks
danicahelb opened this issue Jun 9, 2022 · 4 comments · Fixed by #174 or VEuPathDB/EbrcModelCommon#29
Closed
3 tasks

Study details tab: update summary information header #143

danicahelb opened this issue Jun 9, 2022 · 4 comments · Fixed by #174 or VEuPathDB/EbrcModelCommon#29
Assignees

Comments

@danicahelb
Copy link

danicahelb commented Jun 9, 2022

Final mockups are here: https://www.figma.com/file/mDKIOrpnSwwMjz9acRg3qC/Add-data-accessibility-information-(study-details-%26-downloads-tabs)?node-id=47%3A365

image

To do:

  • Move data accessibility out of "data provenance" and into the summary header section
  • Remove question mark "more information" icon next to "data accessibility"
  • Add in text explaining data accessibility (see below)

The ordering of attributes in the summary section has been moved to a new issue: #175

Text explaining data accessibility:

For users who don’t have access permissions or are not logged in:

For studies with public access:

Data downloads for this study are public. Data is available without logging in.

For studies with controlled access:

To download data please register or log in and request access. Data will be available immediately following request submission.

For studies with protected access:

To download data please register or log in and request access. Data will be available upon study team review and approval.

For users who are logged in and have access to the study data:

For studies with public access:

Data downloads for this study are public. Data is available without logging in.

For studies with controlled access:

You have been granted access to download the data.

For studies with protected access:

You have been granted access to download the data.

Note: request access is a link that should take you to the popup:

image

@sweverschulman
Copy link

Screen Shot 2022-11-04 at 11 10 23 AM

Changes seem to have not been made to the qa site. The ordering is still incorrect and there question mark.

@dmfalke
Copy link
Member

dmfalke commented Nov 4, 2022

It looks like I used the wrong scope for this attribute. I also missed the part about changing the order above. Note that the latter will impact genomic sites, so I think I will make a new issue for that and get feedback from the genomics ux group.

@SheenaTomko
Copy link

This is done

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