-
Notifications
You must be signed in to change notification settings - Fork 31
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
Update to new documentation layout [WD-7866] #323
Conversation
Demo starting at https://anbox-cloud-io-323.demos.haus |
31db39e
to
955361e
Compare
0d916ee
to
6b0939f
Compare
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.
One small suggestion to improve the footer.
Another suggestion to make search results consistent with the docs pages.
86cdf10
to
11ee2b9
Compare
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.
QA looks good now, thanks :)
The scss linter has some errors, can you have a look at it?
c75ef6a
to
759dfcd
Compare
Just added a new commit to fix it, please take a look :) |
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.
Everything LGTM, thanks for the changes and also including the new logo. I think the new logo looks fine.
Tiny nitpick below, then ready for merge from my side.
Let me squash and make the commit history look better |
a4087ad
to
8271f7e
Compare
8271f7e
to
399e646
Compare
Done
QA
Issue / Card
Fixes #
Screenshots