Design review

[meganrm]

• remove hover from row
• remove hover from parental line button
• reduce animation time
• always wrap the snp at the colon
• get rid of semi colon, always have them stacked
• semi bold the numbers on the
• change .disease-catalog-module--banner-content--394e1 to font weight 300
• add dividing line on chart
• fix border radius on blue chart line
• Eliminate extra padding: There is some extra padding under the "Subscribe to our newsletter" line. Eliminate it so that the dividing line to right extends only to the bottom of the text and not beyond. Extend the Coriell button vertically so that it fills to the bottom of the dividing line (which will also be in line with the bottom of the text) #25
• SNP for AICS-104 is incorrect. It should be: NM_000257.4(MYH7):c.752C>A(p.His251Asn)
• AICS-104 should have 2 isogenic controls instead of 1
• All Gene Names should be all lower case (e.g. myosin heavy chain 7) for consistency with current cell catalog
• On some of the parental line buttons it looks like the bottom of the info icon is cut off slightly (can't tell if this is just my screen or not)
• The bottom of each disease table should have rounding on the corners (just realized I failed to do this myself) #26
• Ensure collaborator labs are in italics
• In the very bottom WTC-11 blurb, Genomics should be a hyperlink to: https://www.allencell.org/genomics.html
• Parental Line Modals: make labels (gene symbol, etc.) font size 18. Make content (LMNB1, etc.) font size 16. Make "More Information" font size 16 and semi bolded.
• In each disease table, the column header and each cell line item should be inset 32pt from the left instead of 16pt. Only the table title (Cardiomyopathy, etc.) should be inset 16pt

[meganrm]

@TravisKroeker when this PR finishes building there will be a link to see a working preview

[meganrm]

@TravisKroeker most recent version, which should include all the changes that I've checked off on this issue

[TravisKroeker]

@meganrm These changes look great! A couple notes on the parental line modals:

• "More Info" button text still looks tiny. Bump it up to 16pt and semibold it.
• The changes to the font size in the body of the modal (which look much better) are causing it to wrap. Let's bump up the width of the modal slightly so that it's on a single line. Eventually we might have some gene names that are extraordinarily long and will need to wrap, but not for a couple years if ever.

I might have a few more comments once the obtain cell line and cert. of analysis features are put in place but the rest is soooooo clean.

[meganrm]

@meganrm These changes look great! A couple notes on the parental line modals:

• "More Info" button text still looks tiny. Bump it up to 16pt and semibold it.
• The changes to the font size in the body of the modal (which look much better) are causing it to wrap. Let's bump up the width of the modal slightly so that it's on a single line. Eventually we might have some gene names that are extraordinarily long and will need to wrap, but not for a couple years if ever.

I might have a few more comments once the obtain cell line and cert. of analysis features are put in place but the rest is soooooo clean.

• "More Info" button text still looks tiny. Bump it up to 16pt and semibold it.
• The changes to the font size in the body of the modal (which look much better) are causing it to wrap. Let's bump up the width of the modal slightly so that it's on a single line. Eventually we might have some gene names that are extraordinarily long and will need to wrap, but not for a couple years if ever.

[meganrm]

@TravisKroeker this link has been updated with the two additional tweaks on the parental line modal

[TravisKroeker]

Love it. Noticed you made changes to the icon hover state too to match the text. Good catch.