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

Update UI for data accessibility information on downloads tabs #1172

Closed
danicahelb opened this issue Jun 3, 2022 · 4 comments · Fixed by #1465 or #1485
Closed

Update UI for data accessibility information on downloads tabs #1172

danicahelb opened this issue Jun 3, 2022 · 4 comments · Fixed by #1465 or #1485
Assignees

Comments

@danicahelb
Copy link

Discussed in 6/3/2022 EDA UX meeting. We would like to make improvements to the way the data accessibility information is presented on the downloads tab (which was initially implemented in #1143).

See mockups: https://www.figma.com/file/mDKIOrpnSwwMjz9acRg3qC/Add-data-accessibility-information-(study-details-%26-downloads-tabs)

This ticket needs sign off in a UX meeting before implementing

@danicahelb danicahelb added the UX topic indicates the issue is paused for discussion in EDA UX meeting label Jun 3, 2022
@danicahelb
Copy link
Author

discussed in 6/6/2022 EDA UX meeting.

Have same UI on both the study details tab as on the downloads tab.

Reviewed mockups. Feedback for improvement:

Remove the “more information” question mark as this information is covered in a more straightforward way in the text.

@danicahelb
Copy link
Author

danicahelb commented Jun 9, 2022

We want to mimic the look and feel of the data accessibility information on the study details tab, which is being implemented in VEuPathDB/EbrcWebsiteCommon#143

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:

  • Create a header for "Data Accessibility" using same font styling as for "My Subset" and "Full dataset"
  • Place data accessibility sentence under the "Data Accessibility" header of
  • Update verbiage explaining data accessibility (see below)

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

@nkittur-uga
Copy link
Member

It is looking good except for one small thing: there's a line between the "Study accessibility" row and the text below, which should be removed.

Screen Shot 2022-11-15 at 9 57 08 AM

@SheenaTomko
Copy link
Contributor

the space has been fixed

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