-
Notifications
You must be signed in to change notification settings - Fork 8
DOC-580 | Improve Docs styling and add ArangoGraph Banner #244
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
Conversation
Deploy Preview Available Via |
site/themes/arangodb-docs-theme/static/css/theme-relearn-light.css
Outdated
Show resolved
Hide resolved
site/themes/arangodb-docs-theme/static/css/theme-relearn-light.css
Outdated
Show resolved
Hide resolved
site/themes/arangodb-docs-theme/static/css/theme-relearn-light.css
Outdated
Show resolved
Hide resolved
site/themes/arangodb-docs-theme/static/css/theme-relearn-light.css
Outdated
Show resolved
Hide resolved
The text contrast of inline code is not good, and way too low if it's in a link. The link underline is also not visible below inline code (see " ![]() It doesn't look good and has too little contrast in headlines as well: The copy button overlapping the content is not nice. There would be space to make the code box wider in this case, but to improve it more generally, could we show the copy button only on hover over the code box? ![]() Boxes like here need to be restyled still. Having the same border radius on nested boxes looks wrong: ![]() |
@Simran-B Any further changes should be a part of a separate ticket. It becomes tricky to track changes. Let's keep the scope as-is for this one. Please use this ticket to write down whatever is necessary for the next set of issues: https://arangodb.atlassian.net/browse/DOC-585 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
Description