icgc-argo/argo-docs

Styling fixes after migrating to new docusaurus

Closed this issue · 3 comments

Reviewed the new docusaurus upgrade: https://deploy-preview-460--argo-docs.netlify.app/docs/submission/submission-overview

Also made some style and content changes in this branch to build off of: #463

Some stying items to fix:

  • The chevron is there in the code, but something weird is now happening with the css for width and height I think
    image

  • The way this "Overview" link is coded in the sidebar.js file needs something extra because the sidebar doesn't show on this page: /docs/analysis-workflows/analysis-overview
    Also the bottom "previous/next" links aren't showing up on this page either
    image

image

  • add approx 50px padding/margin between the content and the sidebars (wasn't sure the best way place to put this for responsiveness)

image

  • Can you reduce the size of the h1 - maybe 2.5rem instead of 3rem
    image

  • The header items looked a bit crammed too...compare with production website. Please add a bit of space in between the items (5px on each side of the nav bar items)
    image

@kcullion do you always want 50px padding for the main article col? It's pretty responsive as is.

yeah @ciaranschutte please add 50px extra to these areas. I guess when I referenced responsiveness I meant that I didn't want to break the responsiveness by adding width in the wrong spot (seemed like it was all dynamically calcualted based on screen size)
image

tested this on production, and all styling looks fixed/great now. Thanks @ciaranschutte !