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

Missing comma in list (but maybe change it to <br>s?) #2961

Closed
mccalluc opened this issue Aug 28, 2018 · 1 comment
Closed

Missing comma in list (but maybe change it to <br>s?) #2961

mccalluc opened this issue Aug 28, 2018 · 1 comment
Assignees
Milestone

Comments

@mccalluc
Copy link
Member

There's a missing comma at the end of this line:

    <span ng-if="assay.measurement_accession">
      Measurement Accession: {{ assay.measurement_accession }} <-- here
   </span>

... but maybe change it to <br> for readability? Not worth bike shedding.

@jkmarx
Copy link
Member

jkmarx commented Aug 29, 2018

@mccalluc I'm going to resolve this issue with just a comma now, but good point. The whole details page could use some reformatting. I'm hoping to revisit and enhance the file browser before the end of the year so I will keep this in mind.

@jkmarx jkmarx self-assigned this Aug 31, 2018
@jkmarx jkmarx closed this as completed Aug 31, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants