-
Notifications
You must be signed in to change notification settings - Fork 118
Make sidebar sticky w/signoff #253
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
Make sidebar sticky w/signoff #253
Conversation
Signed-off-by: Ritesh Karankal <ritesh.karankal200@gmail.com>
Yay, your first pull request! 👍 A contributor will be by to give feedback soon. In the meantime, please review the Layer5 Community Welcome Guide and sure to join the community Slack. |
✔️ Website preview ready! 🔨 Explore the source changes: 2061dea 🔍 Inspect the deploy log: https://app.netlify.com/sites/getnighthawk/deploys/6210abebd96d690008f883f8 😎 Browse the preview: https://deploy-preview-253--getnighthawk.netlify.app |
docs/assets/css/sidebar_doc.css
Outdated
} | ||
|
||
.sidebar { | ||
position: absolute; | ||
top: 30px; | ||
position: absolute; |
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.
Het @ritesh-karankal you have to change position of this class, also increase the top also
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.
position: absolute; | |
.sidebar{ | |
position: fixed; | |
top: 145px; | |
Is it correct?
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.
Yeah!! looks good @ritesh-karankal
Signed-off-by: Ritesh Karankal <ritesh.karankal200@gmail.com>
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!! Good Work 👌
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.
@ritesh-karankal I think that formatting change can be ignored (if possible) as it is not required and sometimes it makes PR hard to review, else LGTM
Oh It was due to vscode extension. I will change it back to previous formatting. @Abhijay007 |
Signed-off-by: Ritesh Karankal <ritesh.karankal200@gmail.com>
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 👍 !!
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 🎉 Thanks @ritesh-karankal
Signed-off-by: Ritesh Karankal ritesh.karankal200@gmail.com
Description
This PR fixes #252
Notes for Reviewers
Signed commits