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

Docs QA Items #6300

Closed
sherrylin13 opened this issue Oct 6, 2020 · 1 comment · Fixed by #6467
Closed

Docs QA Items #6300

sherrylin13 opened this issue Oct 6, 2020 · 1 comment · Fixed by #6467
Assignees

Comments

@sherrylin13
Copy link

sherrylin13 commented Oct 6, 2020

  1. Add padding between the copy and the subnav bar throughout - 2
  2. SideNav spacing is too close to the content - 2
  3. Horizontal rules are all missing - see design mocks - 3
  4. Subhead copy links are not in monospace typeface - 2
  5. Edit link is in wrong style (should be like orange arrow link) - 1
  6. Typesafe Invariants is in the wrong typeface, wrong size - 1
  7. GraphQL API section - number indents should align with bullet indents - 2
  8. Mina Protocol Architecture > Lifecycle of a Payment, Consensus -
    Note is in the wrong format (should be in a purple-headed box) - 1
  9. Mina Protocol Architecture > Consensus -
    Under Local_State paragraph needs a period - 3
  10. SNARKs > Which SNARK is right for me? - Illustration for Overview of the Landscape has not been placed - 1
  11. Where there's a purple note with "See" below it, add more space between these 2 elements - 2

-- Mobile --
There is no sticky nav on mobile at all - 1

@aneesharaines aneesharaines changed the title Docs QA Items - Desktop or All Docs QA Items Oct 16, 2020
@aneesharaines
Copy link
Contributor

Combining with #6301

@MartinMinkov MartinMinkov self-assigned this Oct 19, 2020
@MartinMinkov MartinMinkov linked a pull request Oct 21, 2020 that will close this issue
@mergify mergify bot closed this as completed in #6467 Oct 22, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants