Skip to content

Conversation

ritesh-karankal
Copy link
Contributor

Signed-off-by: Ritesh Karankal ritesh.karankal200@gmail.com

Description

This PR fixes #252

Notes for Reviewers

Signed commits

  • Yes, I signed my commits.

Sorry, something went wrong.

Signed-off-by: Ritesh Karankal <ritesh.karankal200@gmail.com>
@welcome
Copy link

welcome bot commented Feb 15, 2022

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.
Be sure to double-check that you have signed your commits. Here are instructions for making signing an implicit activity while peforming a commit.

@netlify
Copy link

netlify bot commented Feb 15, 2022

✔️ 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

}

.sidebar {
position: absolute;
top: 30px;
position: absolute;

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

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
position: absolute;
.sidebar{
position: fixed;
top: 145px;

Is it correct?

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>
Copy link

@YashKamboj YashKamboj left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!! Good Work 👌

@YashKamboj YashKamboj requested a review from debo19 February 16, 2022 14:43
Copy link
Contributor

@Abhijay007 Abhijay007 left a 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

@ritesh-karankal
Copy link
Contributor Author

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>
Copy link
Contributor

@Abhijay007 Abhijay007 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 👍 !!

Copy link
Member

@debo19 debo19 left a 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

@debo19 debo19 merged commit 01d55f0 into layer5io:master Feb 20, 2022
@welcome
Copy link

welcome bot commented Feb 20, 2022

Thanks for your contribution to the Layer5 community! 🎉

Congrats!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Make the sidebar of the docs page sticky.
5 participants