-
Notifications
You must be signed in to change notification settings - Fork 3
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
Study details tab: update summary information header #143
Comments
Most of this work can be done in https://github.com/VEuPathDB/EbrcWebsiteCommon/blob/master/Client/src/components/records/DatasetRecordClasses.DatasetRecordClass.jsx. We will also want to remove the |
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. |
This is done |
Final mockups are here: https://www.figma.com/file/mDKIOrpnSwwMjz9acRg3qC/Add-data-accessibility-information-(study-details-%26-downloads-tabs)?node-id=47%3A365
To do:
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:
For studies with controlled access:
For studies with protected access:
For users who are logged in and have access to the study data:
For studies with public access:
For studies with controlled access:
For studies with protected access:
Note:
request access
is a link that should take you to the popup:The text was updated successfully, but these errors were encountered: